Alexey Petrenko wrote:
2008/2/11, Tim Ellison <[EMAIL PROTECTED]>:
Is this the right place to fix the problem?  Why not teach DRLVM to send
the timezone ID without the trailing newline?
+1

[EMAIL PROTECTED] 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);

If we can fix DRLVM by M5 then I suggest we don't need the workaround at all. Otherwise leave it in but leave the JIRA open and redirect it to DRLVM with a reminder to remove the workaround when fixed properly.

Regards,
Tim


Reply via email to