[Octave-patch-tracker] [patch #8611] weeknum.m for financial package

[Carnë Draug]

Update of patch #8611 (project octave):

                  Status:                    None => Done                   
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #1:

I have pushed your patch to year
http://hg.code.sf.net/p/octave/financial/rev/49ccc42903ed

and the new function
http://hg.code.sf.net/p/octave/financial/rev/e1c98994f646

However, the first broke the tests to the year function (because you should
not reshape if input was a char). It is now fixed
http://hg.code.sf.net/p/octave/financial/rev/f6636bf7a4cb

Also, I made a bunch of changes to the weeknum function to adhere to the
Octave coding guidelines
http://hg.code.sf.net/p/octave/financial/rev/cdd0f6b03a00

If you plan on submitting more patches, would be nice if you could follow the
guidelines. And if you can submit a mercurial changeset/commit, even better ;)

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?8611>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/