RFR-8008118 (original) (raw)
Martin Buchholz martinrb at google.com
Wed Apr 10 18:14:37 UTC 2013
- Previous message: RFR-8008118
- Next message: RFR-8008118
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
I am happy with the latest webrev - ship it! Thanks, John!
On Wed, Apr 10, 2013 at 10:54 AM, John Zavgren <john.zavgren at oracle.com>wrote:
Greetings:
There is a new webrev image available for your inspection that fixes all identified bugs: http://cr.openjdk.java.net/~jzavgren/8008118/webrev.07/ Your comments are welcome. Thanks! John
On 03/29/2013 09:59 AM, christos at zoulas.com wrote: On Mar 28, 8:42pm, martinrb at google.com (Martin Buchholz) wrote: -- Subject: Re: RFR-8008118 | Latest webrev does it this way: | | for (i = 0; i < count; i++) { | char *q = p + strcspn(p, ":"); | pathv[i] = (p == q) ? "." : p; | *q = '\0'; | p = q + 1; | } I prefer to have loops where the sentinel is the actual condition instead of something computed independently, but that's fine too. christos PS: Can we please add strsep(3) to solaris at some point? Everyone else relevant seems to have it.
-- John Zavgrenjohn.zavgren at oracle.com603-821-0904 US-Burlington-MA
- Previous message: RFR-8008118
- Next message: RFR-8008118
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]