[Octave-bug-tracker] [bug #59135] jsondecode converts logical arrays to numeric arrays

[Rik]

Update of bug #59135 (project octave):

                  Status:         Patch Submitted => Fixed                  
             Open/Closed:                    Open => Closed                 

    _______________________________________________________

Follow-up Comment #4:

Thank you Abdallah.  I verified the patch and checked it in here
http://hg.savannah.gnu.org/hgweb/octave/rev/ffa6c50c2eaa.

I made a few small changes.  Octave uses spaces, rather than tabs, because the
tabstop setting is never the same between computers and this always leads to
mis-indented code.  Also, in the commit message one usually writes what was
done and where it was done (function name), rather than just the result.  So I
changed "Fix bug" (result) to "* jsondecode.cc (decode_array_of_arrays): Check
value of is_bool and return
either boolNDArray (logical) or NDArray (numeric)."

I'm marking this bug as fixed and closing it.  Unfortunately, while testing I
found that multi-dimensional logical arrays are encoded to multi-dimensional
numeric arrays.  I'll file a new bug report about that.


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?59135>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/