> The IOCPARM_MASK removal was covered by PSARC case 2008/343.  I'm 
 > thinking that MAXIOCBSZ is not a documented API, isn't used *anywhere* 
 > outside of the files I'm removing it from here (according to Google at 
 > least) -- apart from automatically generated Perl and Python files 
 > (generated from the header files), so I think I can probably get away 
 > without an ARC case for it.  If anyone else on the recipient list feels 
 > different, let me know -- I can file an auto-approval case pretty 
 > trivially.  (Mostly I didn't file the case just to avoid the 1 week 
 > timeout that such normally incurs, particularly since this seems pretty 
 > obvious at this point.)
 > 
 > Webrev is here:
 > 
 > http://cr.opensolaris.org/~gdamore/iocsz

Looks good.  One uber-nit: in strsubr.c, separating the assignment to
`tmp' from the comparison would get rid of the line wrap and thus remove
the need for the braces.

--
meem
_______________________________________________
opensolaris-code mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/opensolaris-code

Reply via email to