Re: [PATCH 16/16] cal: simplify day_in_week()

Sami Kerola
2013-05-03T20:19:05Z

On 2 May 2013 19:51, Sami Kerola <kerolasa@iki.fi> wrote:
> The day_in_week() does not need to perform fullfledged September 1752
> calculations, that month is dealt as static assignment.

Hi Karel, and others,

Please drop this patch. Having abort() at the end of the function is
bad choice, if something like weekday name highlighting is
implemented[1]. When I reconsider that bit the function it should
return for the eleven days at September 1752 a day out of bounds,
which basically some sort of NaN equivalent.

[1] http://www.spinics.net/lists/util-linux-ng/msg07904.html

--
Sami Kerola