search

gdamore / illumos-core

Community developed and maintained version of the OS/Net consolidation
http://www.illumos.org/projects/illumos-gate
14 stars 3 forks source link

ppoll() incorrectly listed as Standard #154

Closed gdamore closed 9 years ago

gdamore commented 9 years ago

While merging in most of Joyent's man page changes (most of which are inferior to the work I'd already done here unfortunately), I noticed ppoll() was incorrectly listed as Standard.

For the most after careful review of the deltas, I've decided to keep my changes in preference to Joyent's for a couple of reasons. First I was more careful about documenting standards compliance with respect to POSIX 2008. Second, I updated various old pages to mdocml, whereas Joyent left them in man(5) format. Third I noticed numerous minor spelling errors, and in some cases inferior wording. I also kept futimens() and utimensat() in a separate page from the legacy utimes() and futimesat() page.

Where Joyent added new content, as in cv_reltimed_wait, etc. I've kept those changes. I've also fixed one bug that the Joyent changes uncovered in my changes, which is that I forgot to link fchmodat() properly.

gdamore commented 9 years ago

To be clear, the only bug I'm listing here is the ppoll() changes.

And this will need to be fixed after finishing the merge with upstream.

gdamore commented 9 years ago

I also corrected some of the wording relating to timeouts. Specifically, the ppoll() function uses nanosecond timeouts for waiting, and struct timespec tsp, rather than timeout. And I added details about standards conformance (basically poll() is available in all environments, and ppoll() in none.)