Re: [PATCH] ISO weeks for calendar

>>>>> Michael wrote: Michael> several moons ago I hacked up calendar mode to display the Michael> ISO or business weeks within the calendar. ISO weeks is a good thing. I have been wanting them too. Nice that you have given it a try. Michael> Perhaps some kind soul (who has deeper lisp knowledge than I Michael> have) may find this useful and can do a review of this piece Michael> of code. I don't know about deep lisp knowledge but I found one or possibly two problems when I tried the patch. ====================================================================== * 1 -- The function calendar-iso-from-absolute is not loaded. Added an autoload for that.

====================================================================== * 2 -- The week numbers are accepted as valid dates causing insert-diary-entry to behave odd. Should be "Not a valid date" just as if doing the same on other bad positions in the buffer I think. This is what I came up with -- Ignore the match if in iso week columns. Please check and improve. (Partial patch: Just the defun calendar-insert-indented included.)

Michael> I hope to see it in the calendar package someday. This patch looks promising for the upstream version of calendar. You might want to try to get it in there.