"Roger A. Faulkner" wrote:
> 
> I am sponsoring this automatic case for myself.
> 
> 1.  Introduction
> 
>     This case adds a new posix_spawn() support function to the C library:
> 
>        int posix_spawn_file_actions_addclosefrom_np(
>            posix_spawn_file_actions_t *file_actions,
>            int filedes);
> 
>     The committment level of this interface is Committed.
>     The "_np" suffix indicates that this is a non-portable
>     Solaris extension.

Two nits:
- IMO it may be nice to use a different suffix than "_np", e.g. IMO
|posix_spawn_file_actions_addclosefrom_solaris()| may be better.
- What about adding a |flags| argument to allow that future extensions
can be made, e.g. change the prototype to...
-- snip --
int posix_spawn_file_actions_addclosefrom_solaris(
            posix_spawn_file_actions_t *file_actions,
            int filedes,
            uint32_t flags
            )
-- snip --

>     The release binding is "minor release" so it can go
>     into SunOS 5.11 (OpenSolaris) (there is no need or
>     intention to back-port the interface to SunOS 5.10).

Why ? It may be _nice_ to have it in Solaris 10 on demand, too (e.g. you
don't have to do the backport now but it keeps the option open to do a
backport to Solaris 10 without another ARC case).

----

Bye,
Roland

-- 
  __ .  . __
 (o.\ \/ /.o) roland.mainz at nrubsig.org
  \__\/\/__/  MPEG specialist, C&&JAVA&&Sun&&Unix programmer
  /O /==\ O\  TEL +49 641 3992797
 (;O/ \/ \O;)

Reply via email to