Alexei Zakharov wrote:
FYI I've marked the fix as a "workaround" in JIRA. I'll create new
DRLVM issue with detailed description of the problem. I promise to
roll back this workaround as soon as we have this issue fixed in
DRLVM. Does this make sense?
Sure
Thanks,
Tim
2008/2/11, Tim Ellison <[EMAIL PROTECTED]>:
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
