[Octave-bug-tracker] [bug #33993] __xzip__ with nargout = 0, nargin = 4 does not move compressed files

[Jordi Gutiérrez Hermoso]

Update of bug #33993 (project octave):

                  Status:                    None => Fixed                  
             Open/Closed:                    Open => Closed                 
                 Summary: __xzip__ with nargout = 0, nargin = 4 is broken =>
__xzip__ with nargout = 0, nargin = 4 does not move compressed files

    _______________________________________________________

Follow-up Comment #1:

Thanks, I've pushed your fix on the stable branch after confirming that it is
indeed a regression in expected behaviour:

     http://hg.savannah.gnu.org/hgweb/octave/rev/5cc3b7673d25

In the future, if you would like to submit more patches, it would be
preferrable if you could do them with Mercurial:

     http://jordi.inversethought.com/blog/how-to-write-a-patch-for-octave/
    
http://www.gnu.org/software/octave/doc/interpreter/How-to-Contribute.html#How-to-Contribute

Regarding unit testing, we do in fact need to write more. If you could provide
tests, that would be great. We will be having our next code sprint during
September 3 in order to address this precise problem of writing unit tests for
all of Octave's functions, or as many as we can get done. Of course, you don't
have to wait until then to provide tests. ;-)

Thanks for you attention to detail and for providing fixes for them.

    _______________________________________________________

Reply to this item at:

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

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