Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RestartKey has to be supplied, *RestartKey may be NULL. #1594

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from

Conversation

rodwiddowson
Copy link
Contributor

This will need changed to be correct markup

The documentation says a lot of things like

 If RestartKey is NULL,

When it should say

 If *RestartKey is NULL

or (in a more ugly manner)

 Of the address pointed to by RestartKey is NULL

If you actually to specify the parameter as NULL things go sidesways pretty fast and empirically specifying a restart key which points to NULL does what is described in the doco.

This PR tried to capture that in a couple of places. Its wrong because I dunno how to specify a * in a italicized chunk of code.

I may have missed some places to

*This will need changed to be correct markup*

The documentation says a lot of things like

     If RestartKey is NULL,

When it should say

     If *RestartKey is NULL

or (in a more ugly manner)

     Of the address pointed to by RestartKey is NULL

If you actually to specify the parameter as NULL things go sidesways pretty fast and empirically specifying a restart key which points to NULL does what is described in the doco.

This PR tried to capture that in a couple of places. Its wrong because I dunno how to specify a * in a italicized chunk of code.

I may have missed some places to
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants