bugGNU Octave - Bugs: bug #38290, inferiorto() doesn'' behave as...

 
 

bug #38290: inferiorto() doesn'' behave as expected

Submitter:  Julien Bect <jbect>
Submitted:  Sat 09 Feb 2013 10:25:21 AM UTC
   
 
Category:  Interpreter 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
   

Jump to the original submission

Sat 23 Feb 2013 06:43:12 PM UTC, comment #8: 

I reformatted your commit message to word-wrap at 80 columns and I rebased it:

http://hg.savannah.gnu.org/hgweb/octave/graph/c81b4c81339f

You may wish to remove your local copy of this commit with


hg strip -r e164fefa0d425


Jordi GutiƩrrez Hermoso <jordigh>
Group Member
Sat 23 Feb 2013 12:34:05 PM UTC, comment #7: 

The new test classes added by the patch are missing from test/classes/module.mk. Here is a patch to fix this:

(file #27510)

Julien Bect <jbect>
Thu 21 Feb 2013 08:02:56 AM UTC, comment #6: 

Great. Could you please close patch #7946 ?

https://savannah.gnu.org/patch/?7946

Julien Bect <jbect>
Wed 20 Feb 2013 08:43:43 PM UTC, comment #5: 

I have applied your patch, along with minor whitespace style fixes:

http://hg.savannah.gnu.org/hgweb/octave/graph/ac672925fc98

Thanks.

Jordi GutiƩrrez Hermoso <jordigh>
Group Member
Sun 10 Feb 2013 10:33:46 AM UTC, comment #4: 

I have rebased the patch to cset:999f8257313b (test_classes.m renamed to classes.tst) and run a full "make check". Everything still looks ok.

(file #27437)

Julien Bect <jbect>
Sat 09 Feb 2013 05:48:22 PM UTC, comment #3: 

A file was missing from the patch, here it is.

(file #27427)

Julien Bect <jbect>
Sat 09 Feb 2013 01:00:53 PM UTC, comment #2: 

The attached patch fixes the bugs in inferiorto().

Several other functions related to class precedence are rewritten for clarity, and tests are added to test/classes/test_classes.m.

[Note: this was initially submitted as patch #7946]

(file #27426)

Julien Bect <jbect>
Sat 09 Feb 2013 10:28:17 AM UTC, comment #1: 

I meant "doesn'T behave as expected" and "fails silentLy"... The worse part is, I actually proofread my post before sending... Is there any way I can correct my post after sending it ?

Julien Bect <jbect>
Sat 09 Feb 2013 10:25:21 AM UTC, original submission:  

The attached tests exhibit two bugs in inferiorto().

First bug: calling inferiorto() when the opposite precendence is already set fails silenty (neither does it generate an error, as expected, nor does it actually change the precedence relationship).

Second bug: calling inferiorto() when no precedence is already set generates an undue error.

Both are actually related to the same error in the C++ code. A patch with bugfix & tests will follow shortly.

Julien Bect <jbect>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #27437:  jbFixPrecedenceBug7.patch added by jbect (12KiB - text/x-patch)
file #27427:  jbFixPrecedenceBug5.patch added by jbect (12KiB - text/x-patch)
file #27426:  jbFixPrecedenceBug4.patch added by jbect (12KiB - text/x-patch)
file #27425:  showPrecedenceBugs.zip added by jbect (11KiB - application/zip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by jordigh (Posted a comment)
  • -email is unavailable- added by jbect (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 7 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-02-23 jbect Attached File- Added jbFixTestClassesModuleDotMk.patch, #27510
    2013-02-20 jordigh StatusNone Fixed
        Open/ClosedOpen Closed
    2013-02-10 jbect Attached File- Added jbFixPrecedenceBug7.patch, #27437
    2013-02-09 jbect Attached File- Added jbFixPrecedenceBug5.patch, #27427
    2013-02-09 jbect Attached File- Added jbFixPrecedenceBug4.patch, #27426
    2013-02-09 jbect Attached File- Added showPrecedenceBugs.zip, #27425

    Back to the top

    Powered by Savane 3.13-3230.
    Corresponding source code