<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Sat, 21 Mar 2026 at 01:53, Zoltan Somogyi <<a href="mailto:zoltan.somogyi@runbox.com">zoltan.somogyi@runbox.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
On Fri, 20 Mar 2026 23:32:25 +1100, Julien Fischer <<a href="mailto:jfischer@opturion.com" target="_blank">jfischer@opturion.com</a>> wrote:<br>
> On Fri, 20 Mar 2026 at 21:17, Zoltan Somogyi <<a href="mailto:zoltan.somogyi@runbox.com" target="_blank">zoltan.somogyi@runbox.com</a>><br>
> wrote:<br>
> > Almost all of that is good, so commit it. I don't particularly like<br>
> > either the documentation or the implementation of the leap year test,<br>
> > but it is easier for me to make changes and post them for review<br>
> > than to describe them.<br>
> <br>
> Committed.<br>
<br>
My proposal is attached. The main point is to avoid nested exceptions<br>
in the description, and making the code follow the description directly.<br>
N mod 25 is not faster than N mod 100, so I don't see how its negative<br>
effect on readability would be worth it.<br></blockquote><div><br></div><div>That's fine. It uses one more rem than the previous version, but since</div><div>that's down a branch that will rarely be executed that's alright.</div><div><br></div><div>Julien.<br></div></div></div>