[Octave-bug-tracker] [bug #51219] Typos in bicgstab.m cgs.m (scripts/sparse)

[Carlo de Falco]

Update of bug #51219 (project octave):

             Open/Closed:                  Closed => Open                   

    _______________________________________________________

Follow-up Comment #4:

I am not really sure the same exact bug exists in the
new version of the functions, as in the new version 
the conversion of all types of matrix inputs is handled
by a single function for all solvers:

https://bitbucket.org/cdf1/octave/src/06f7d173f2f5231c05c44b13ff0619b4f4eb9d0a/scripts/sparse/private/__alltohandles__.m?at=dorigo&fileviewer=file-view-default

And the code for this looks correct to me there.

Maybe this is a good time to make the merge?

I have been maintaining this patch in a branch named "dorigo",

I just tried the merge in my repository on bitbucket and it
goes very smoothly, changes are only made to m-files so this
should be a relatively non-invasive merge although quite big.

If someone would like to review the changes a full diff can be seen via this
link:

https://bitbucket.org/cdf1/octave/branches/compare/dorigo%0Ddefault#diff

c.


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?51219>

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