bugGNU Octave - Bugs: bug #59398, [octave forge] (sparsersb) Invalid...

 
 

bug #59398: [octave forge] (sparsersb) Invalid empty indexing

Submitter:  Markus Mützel <mmuetzel>
Submitted:  Mon 02 Nov 2020 10:52:50 AM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * other Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 02 Nov 2020 04:24:05 PM UTC, comment #5: 

Thanks. Looks good to me.

Since this is more like an internal function that a regular user wouldn't use, I don't think it is necessary to cherry pick this patch in MXE Octave. Waiting for the next upstream release is Good Enough (TM) imho.

Closing as fixed.

Markus Mützel <mmuetzel>
Group administrator
Mon 02 Nov 2020 04:04:39 PM UTC, comment #4: 

The results match.
Substituting those f{} with f{:} fixes the problem on my side; so I guess the test passes on your side now.
I patched upstream sparsersb this way http://hg.code.sf.net/p/octave/sparsersb//rev/20ec361e0470
:-)

dezperado <dezperado>
Mon 02 Nov 2020 03:25:17 PM UTC, comment #3: 

Attaching the diary when I ran `sparsersbtg()` after that change in Octave 6 on Windows.

(file #50194)

Markus Mützel <mmuetzel>
Group administrator
Mon 02 Nov 2020 02:29:14 PM UTC, comment #2: 

Thanks for the hint; I don't have Octave 6.
Can you please substitute the four occurrences of 'f{}' with 'f{:}' and run:


octave --eval 'sparsersbtg()'

...then what output do you get ?
I get the attached one, in both cases.

(file #50193)

dezperado <dezperado>
Mon 02 Nov 2020 10:57:01 AM UTC, comment #1: 

Probably the same for the string in line 103.

Markus Mützel <mmuetzel>
Group administrator
Mon 02 Nov 2020 10:52:50 AM UTC, original submission:  

Running the following command (part of the BISTs), fails with an error:

>> length( sparsersbtg () )
error: invalid empty index expression
error: called from
    sparsersbtg at line 96 column 8


Empty indexing is invalid in Octave 6 and later.
`f{}` should probably be replaced by `f{:}` (twice) in the indicated line.

CC'ing Michele Martone as the package maintainer.

Markus Mützel <mmuetzel>
Group administrator

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #50194:  sparsersbtg.log added by mmuetzel (84KiB - application/octet-stream)
file #50193:  sparsersbtg.out added by dezperado (82KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dezperado (Updated the item)
  • -email is unavailable- added by mmuetzel (Submitted the item)
  • -email is unavailable- added by mmuetzel
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only group members can vote.

     

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-11-02 mmuetzel StatusNone Fixed
        Open/ClosedOpen Closed
    2020-11-02 mmuetzel Attached File- Added sparsersbtg.log, #50194
    2020-11-02 dezperado Attached File- Added sparsersbtg.out, #50193
    2020-11-02 mmuetzel Carbon-Copy- Added dezperado

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code