Review request #1: 6863566 (Java should support the freedesktop.org startup notification specification) (original) (raw)
Alan Bateman Alan.Bateman at Sun.COM
Thu Oct 15 13:47:56 UTC 2009
- Previous message: Review request #1: 6863566 (Java should support the freedesktop.org startup notification specification)
- Next message: Review request #1: 6863566 (Java should support the freedesktop.org startup notification specification)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Martin, Michael - have either of you looked at the filtering that they are proposing to add to ProcessEnvironment?
Anthony Petrov wrote:
Hello,
Please review the next version of the fix contributed by Damjan Jovanovic: RFE: https://bugs.openjdk.java.net/showbug.cgi?id=100094 There you can also find some latest comments regarding the fix. webrev: http://cr.openjdk.java.net/~anthony/7-24-startupNotify-6863566.1/ Since the patch includes changes to the src/solaris/classes/java/lang/ProcessEnvironment.java, I'm CC'ing the Core Libs alias to review the changes in that file.
From AWT perspective, the changes look good to me. -- best regards, Anthony
- Previous message: Review request #1: 6863566 (Java should support the freedesktop.org startup notification specification)
- Next message: Review request #1: 6863566 (Java should support the freedesktop.org startup notification specification)
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]