Yes, looks fine to me too. I would just update the comment above this
code to add Mac OS to the Solaris case.
Thanks
Michael
On 13/01/12 21:02, Kurchi Hazra wrote:
How does this look: http://cr.openjdk.java.net/~khazra/7127771/webrev.01/
- Kurchi
On 1/13/2012 12:14 PM, Alan Bateman wrote:
Bug : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7127771
Webrev : http://cr.openjdk.java.net/~khazra/7127771/webrev.00/
What you have is fine although you could combine with the Solaris
code? Should the __ALLBSD_SOURCE XXX be removed while you are there?
-Alan
