bugGNU Octave - Bugs: bug #39282, symbolic package: Proper reversal...

 
 

bug #39282: symbolic package: Proper reversal of a cell array

Submitter:  Rafael Laboissière <rlaboiss>
Submitted:  Wed 19 Jun 2013 08:59:42 AM UTC
   
 
Category:  Octave Package Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Incorrect Result
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * GNU/Linux
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 19 Jun 2013 11:08:19 PM UTC, comment #3: 

Unfortunately, the symbolic package does not have a maintainer at all.

Carnë Draug <carandraug>
Group Member
Wed 19 Jun 2013 08:58:51 PM UTC, comment #2: 

The reversed function existed in a old version of Octave (2.1, at least) and accepted a list as argument.  This is an indication of how badly the symbolic package has been maintained.

Rafael Laboissière <rlaboiss>
Wed 19 Jun 2013 07:34:26 PM UTC, comment #1: 

That's an interesting bug. I wonder where does reverse came from, it doesn't exist in matlab either. Maybe from a very old Octave version.

Anyway, I applied the changeset after some very small whitespace change. I know this function doesn't really follow the typical convention, but we usually prefer to have no spaces when indexing a variable. The spaces are only used for functions. Also, a very small change to the commit message (I'm trying to make Octave Forge follow Octave core commit guidelines).

Carnë Draug <carandraug>
Group Member
Wed 19 Jun 2013 08:59:42 AM UTC, original submission:  

The "reverse" function is called in inst/sym2poly.m in order to reverse the cell array c_ex.  This function does not exist in Octave.  Use indexing with parenthesis, instead.

The changeset attached to this bug report fixes the problem.

Rafael Laboissière <rlaboiss>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #28347:  cell-array-reversal.diff added by rlaboiss (746B - text/x-patch - Mercurial changeset for inst/sym2poly.m)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rlaboiss (Submitted the item)
  •  

    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 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-06-19 carandraug StatusNone Fixed
        Open/ClosedOpen Closed
    2013-06-19 rlaboiss Attached File- Added cell-array-reversal.diff, #28347

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code