RFR-8008118
Christos Zoulas
christos at zoulas.com
Wed Mar 27 01:58:19 UTC 2013
On Mar 26, 6:13pm, martinrb at google.com (Martin Buchholz) wrote:
-- Subject: Re: RFR-8008118
| Here is my alternative set of perfectionist changes:
:-)
| # Fix PATH handling
| http://cr.openjdk.java.net/~martin/webrevs/openjdk8/pathv/
LGTM, only question is why error from one allocation throws and from the
other does not? I would move the throw after the:
if (splitpath == NULL || pathv == NULL) {
This way we don't need to pass env to xstrdup(). Now you are going to make
me grep on how many xstrdup()'s are in the tree :-)
Correct and simple otherwise.
christos
More information about the core-libs-dev
mailing list