Skip to content

Conversation

@sethp-nr
Copy link
Contributor

As per e.g. https://www.freedesktop.org/software/systemd/man/sd_journal_get_realtime_usec.html:

Note that these functions will not work before sd_journal_next(3) (or related call) has been called at least once, in order to position the read pointer at a valid entry.

The prior wording of the documentation was ambiguous regarding its usage of "current". This change attempts to clarify that it's "current" in terms of "the state of the read pointer, adjusted by the Next/Previous family of functions" vs. "current" in terms of chronology.

As per e.g. https://www.freedesktop.org/software/systemd/man/sd_journal_get_realtime_usec.html:

> Note that these functions will not work before sd_journal_next(3) (or related call) has been called at least once, in order to position the read pointer at a valid entry.

The prior wording of the documentation was ambiguous regarding its usage of "current". This change attempts to clarify that it's "current" in terms of "the state of the read pointer, adjusted by the Next/Previous family of functions" vs. "current" in terms of chronology.
@lucab lucab changed the title Clarify documentation's notion of "current" sdjournal: clarify documentation's notion of "current" Nov 30, 2017
Copy link
Contributor

@lucab lucab left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucab lucab merged commit c966a8a into coreos:master Nov 30, 2017
@sethp-nr sethp-nr deleted the patch-1 branch November 30, 2017 16:07
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