<br><br><div class="gmail_quote">2008/11/13 Aaron J. Seigo <span dir="ltr"><<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Tuesday 11 November 2008, leo peng wrote:<br>
> Chinese environment, run "kcmshell4 clock", "Indian / Christmas" and<br>
> "Pacific / Kiritimati" have the same city name at zone list.<br>
> so time zone list error.<br>
<br>
</div>not that it particularly matters to the algorithm in question, but these two<br>
cities don't have this problem here. still ... i can see it occuring, yes.<br>
<div class="Ih2E3d"><br>
> Time zone list of the index should be "city +<br>
> Time zone" in kdelibs/kdeui/widgets/ktimezonewidget.cpp.<br>
><br>
> I add a patch, please see attachment.<br>
<br>
</div>the second part of the patch looks ok, but the first part looks wrong.<br>
<br>
from reading it, it looks a bit flawed, or at least unecessarily inneficient.<br>
essentially you want Europe/Berlin to become BerlinEurope, right?<br>
<br>
in which case ... wouldn't this be easier, more efficient and guaranteed to<br>
always work:<br>
<br>
int separator = contentCity.lastIndexOf('/');<br>
QString city = continentCity.right(length - separator - 1) +<br>
continentCity.left(separator);<br>
<br>
?<br>
<font color="#888888"><br>
--<br>
Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43<br>
<br>
KDE core developer sponsored by Qt Software<br>
<br>
</font><br><br>
>> Visit <a href="http://mail.kde.org/mailman/listinfo/kde-devel#unsub" target="_blank">http://mail.kde.org/mailman/listinfo/kde-devel#unsub</a> to unsubscribe <<<br>
<br></blockquote></div><br>I change my first patch,  and fix this problem, please see attachment.<br>Thank you for your help,  Aaron J.Seigo.<br>Thank you for your help, Albert.<br clear="all"><br>-- <br>  best regards !<br>
----------------------<br>leopeng<br>