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

 
 

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

Submitted by:  Rafael Laboissiere <rlaboiss>
Submitted on:  Wed 19 Jun 2013 08:59:42 AM UTC  
 
Category: Octave Forge PackageSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project 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 Laboissiere <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>
Project 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 Laboissiere <rlaboiss>

 

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

Attach File(s):
   
   
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
  • -unavailable- added by rlaboiss (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only project members can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 19 Jun 2013 07:34:26 PM UTCcarandraugStatusNone=>Fixed
      Open/ClosedOpen=>Closed
    Wed 19 Jun 2013 08:59:42 AM UTCrlaboissAttached File-=>Added cell-array-reversal.diff, #28347

    Back to the top


    Powered by Savane 3.1-cleanup1