harmony-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alexey Petrenko" <alexey.a.petre...@gmail.com>
Subject Re: svn commit: r620477 - /harmony/enhanced/classlib/trunk/modules/luni/src/main/java/java/util/TimeZone.java
Date Mon, 11 Feb 2008 13:15:24 GMT
2008/2/11, Tim Ellison <t.p.ellison@gmail.com>:
> Is this the right place to fix the problem?  Why not teach DRLVM to send
> the timezone ID without the trailing newline?
+1

> ayza@apache.org wrote:
> > +        if (zone.contains("\n")) {
> > +            zone = zone.substring(0, zone.indexOf("\n"));
> > +        }
As far as I understood the string ends with "\n".
If so I believe it's better use the following construction here:
if (zone.endsWith("\n"))
    zone = zone.sunstring(0, zone.length() -1);

SY, Alexey

Mime
View raw message