On 26 October 2012 15:24,  <littleman...@yahoo.com> wrote:
> Hi Carnë,
>
> After more careful inspection, it appears that when New Years Day falls on a 
> Saturday, the holiday is not observed at all. The holidays function does not 
> check that, so modifying line 41 will fix that (and also allow holidays to 
> pass its test). So please disregard the previous code change suggestion, and 
> consider the following instead.
>
> from holidays.m:
> ------------------------------------------
>   ## New Year's Day
>   tmphol = datenum (yrs, 1, 1);
>   hol = [hol; tmphol(weekday(tmphol(:)) ~= 7)(:)];
> ------------------------------------------
> I modified the last line from
>   hol = [hol; tmphol(:)];
>
> Thanks for the link. I will submit octave-forge bugs there in the future. 
> Should I open up a new thread there for this bug, or is that unnecessary?
>
> -Randy Chamberlin

Hi Randy

not necessary this time. I have commited your changes see
https://sourceforge.net/p/octave/code/11368

Thank you,
Carnë

------------------------------------------------------------------------------
WINDOWS 8 is here. 
Millions of people.  Your app in 30 days.
Visit The Windows 8 Center at Sourceforge for all your go to resources.
http://windows8center.sourceforge.net/
join-generation-app-and-make-money-coding-fast/
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to