<div dir="auto">I'm not so sure. at the very least you will need to change the comment because it specifically states this isn't what should be done. <div dir="auto"><br></div><div dir="auto">when I get back from the water polo game I'll give a better argument (or retract my statement).</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Oct 17, 2018, 17:43 David T. Lewis <<a href="mailto:lewis@mail.msen.com">lewis@mail.msen.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">If I am reading this right, it says that we can test for<br>
"self start = comparand start" if and only if both of the two Timespans<br>
have no timezone information, otherwise we need to use #hasEqualTicks:<br>
to compare the two start values for the two durations.<br>
<br>
I'm not sure if there is an optimization available for the case of the<br>
two Timespans both having timezone information, but aside from that<br>
the change looks right to me.<br>
<br>
Dave<br>
<br>
On Wed, Oct 17, 2018 at 09:04:29PM +0000, <a href="mailto:commits@source.squeak.org" target="_blank" rel="noreferrer">commits@source.squeak.org</a> wrote:<br>
> Chris Muller uploaded a new version of Chronology-Core to project The Inbox:<br>
> <a href="http://source.squeak.org/inbox/Chronology-Core-cmm.13.mcz" rel="noreferrer noreferrer" target="_blank">http://source.squeak.org/inbox/Chronology-Core-cmm.13.mcz</a><br>
> <br>
> ==================== Summary ====================<br>
> <br>
> Name: Chronology-Core-cmm.13<br>
> Author: cmm<br>
> Time: 17 October 2018, 4:04:08.832696 pm<br>
> UUID: 60718249-84a8-4dc2-aa94-4ba5c8e5addc<br>
> Ancestors: Chronology-Core-tcj.12<br>
> <br>
> Fix DateAndTime today asDate = Date today even when not in GMT.<br>
> <br>
> =============== Diff against Chronology-Core-tcj.12 ===============<br>
> <br>
> Item was changed:<br>
>   ----- Method: Timespan>>= (in category 'ansi protocol') -----<br>
>   = comparand<br>
>       ^ self class = comparand class <br>
> +             and: [((self noTimezone and: [comparand noTimezone])<br>
> -             and: [((self noTimezone or: [ comparand noTimezone ])<br>
>                       ifTrue: [ self start hasEqualTicks: comparand start ]<br>
>                       ifFalse: [ self start = comparand start ])<br>
>               and: [ self duration = comparand duration ] ]<br>
>   .!<br>
> <br>
> <br>
<br>
</blockquote></div>