Details
Details
- Reviewers
emaste - Group Reviewers
manpages - Commits
- rS304287: Add fdatasync(2) man page, combined with fsync(2).
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Skipped - Unit
Tests Skipped
Event Timeline
lib/libc/sys/fsync.2 | ||
---|---|---|
78 | Passive -> active: If the file metadata has already been committed, using | |
81 | "for the same purpose" is a little unclear. How about can be more efficient than .Fn fsync . | |
95 | Either remove the "The" at the beginning of this sentence, or add a word before "fail" to say what they are: "fsync and fdatasync fail if:" |
Comment Actions
Other than the optional change I suggested for the '.Nd' line, this looks good to me.
lib/libc/sys/fsync.2 | ||
---|---|---|
42 | "synchronize changes to a file's data, without or with it's metadata" |
lib/libc/sys/fsync.2 | ||
---|---|---|
42 | Wouldn't it clutter the tag line ? Some changes are synced, which are is described in the main text. fdatasync(2) cannot be a default choice, it requires careful consideration if used. |
lib/libc/sys/fsync.2 | ||
---|---|---|
37 | Update |