8034856/8034857: More gcc warnings (original) (raw)

Mikael Vidstedt mikael.vidstedt at oracle.com
Mon Feb 17 19:59:04 PST 2014

On 2014-02-17 07:08, Alan Bateman wrote:

On 17/02/2014 05:51, Mikael Vidstedt wrote:

I'm inclined to agree with this. Since the code depends on a specific behavior of isspace which does not match what the system provided function does I too think it would be more robust to implement our own version of it. I completely agree that changing this code to use its own isspace is the right thing, it just seems a bit much for a drive-by fixed to gcc warnings. Do either of you want to take it?

How about:

http://cr.openjdk.java.net/~mikael/webrevs/isspace/webrev.00/webrev/

Cheers, Mikael

-------------- next part -------------- An HTML attachment was scrubbed... URL: http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20140217/0cf3246b/attachment.html

More information about the serviceability-dev mailing list