On 7/30/2013 5:24 PM, Alan Bateman wrote:
On 29/07/2013 10:06, Alexey Utkin wrote:
Thanks, Alan
I did "if" reformatting and small refactoring for the code in
function in accordance with
https://jbs.oracle.com/bugs/browse/JDK-5008166
recommendations. I am sure that the presence of path separator has no
dependence from search direction.
Here is the webrev:
http://cr.openjdk.java.net/~uta/openjdk-webrevs/JDK-8021245/webrev.01/
This looks much nicer so thumbs up from me. One thing just to say that
I don't think the JDK has been supported for Windows 9x/ME for several
years so one of the SearchPaths isn't used nowadays.
The [selectProcessFlag] function is obsolete. But the code needs to be
clean.
Minor comment is that the new comments probably should a space
before/after the /* */ to be consistent with the other comments (no
need to generate a new webrev for that of course).
Accepted.
-Alan
