David, Thank you for review. Nit fixed in-place.
PS: I'd followed Alan Coopersmith blog https://blogs.oracle.com/alanc/entry/solaris_11_2_functional_deprecation So it looks like this time door_cred() is really deprecated :) -Dmitry On 2014-07-26 14:59, David Holmes wrote: > Hi Dmitry, > > On 26/07/2014 7:46 PM, Dmitry Samersoff wrote: >> Please, >> >> Review the small fix. >> >> http://cr.openjdk.java.net/~dsamersoff/JDK-8049046/webrev.01/ >> >> door_cred() is deprecated since Solaris 11.2 so replace it to >> door_ucred() > > 11.2? door_cred has been marked obsolete since before that file started > using it (if we can believe the copyright date)! :) > > The door_cred() function is obsolete. Applications should > use the door_ucred(3DOOR) function in place of door_cred(). > > ATTRIBUTES > See attributes(5) for descriptions of the following attri- > butes: > > SunOS 5.10 Last change: 25 Mar 2003 1 > ------------- > > > Change looks okay to me. A minor style nit - I dislike int variables > that pretend to be booleans, I'd prefer > > int ok > > be simply > > int ret > > Cheers, > David > ------ > >> Testing: >> >> JPRT >> -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.
