RFR: JDK-8026126 and others - Solaris and Linux code cleanup

[Daniel D. Daugherty]

 > http://cr.openjdk.java.net/~coleenp/linux-cleanup/

src/os/linux/vm/os_linux.cpp
     No comments.

src/os/linux/vm/perfMemory_linux.cpp
     No comments.


 > http://cr.openjdk.java.net/~coleenp/solaris-cleanup/

src/os/solaris/vm/os_solaris.cpp
     No comments.

src/os/solaris/vm/perfMemory_solaris.cpp
     No comments.

src/os_cpu/solaris_x86/vm/os_solaris_x86.cpp
     No comments.

Thumbs up.

I presume you got no push back on dropping the assert() for
the return value from ::close().

Dan


On 2/5/14 7:47 AM, Gerald Thornbrugh wrote:
> Hi,
>
> I am a member of the Oracle Hotspot group and there is an initiative 
> to clean up
> things like uninitialized variables and unclosed file descriptors in 
> JDK9.  I have run
> UTE and JPRT tests on Solaris and Linux test machines. The below 
> webrevs show
> the cleanup in some of the Solaris and Linux source areas.
>
> The Linux cleanup changes:
>
> Bugs:
> https://bugs.openjdk.java.net/browse/JDK-8026126
> https://bugs.openjdk.java.net/browse/JDK-8025497
>
> Webrev:
>
> http://cr.openjdk.java.net/~coleenp/linux-cleanup/
>
> The Solaris cleanup changes:
>
> Bugs:
> https://bugs.openjdk.java.net/browse/JDK-8025481
> https://bugs.openjdk.java.net/browse/JDK-8029777
> https://bugs.openjdk.java.net/browse/JDK-8025665
>
> Webrev:
> http://cr.openjdk.java.net/~coleenp/solaris-cleanup/
>
> Please review my changes and let me know if you have questions.
>
> Thanks,
>
> Gerald Thornbrugh