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


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

Submitted by:  Julien Bect <jbect>
Submitted on:  Sat Feb 9 10:25:21 2013  
Category: InterpreterSeverity: 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 MarkupRich Markup):

You are not logged in

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


(Jump to the original submission Jump to the original submission)

Sat Feb 23 18:43:12 2013, comment #8:

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


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

Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Sat Feb 23 12:34:05 2013, 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 Feb 21 08:02:56 2013, comment #6:

Great. Could you please close patch #7946 ?


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

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



Jordi GutiƩrrez Hermoso <jordigh>
Project Administrator
Sun Feb 10 10:33:46 2013, 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 Feb 9 17:48:22 2013, comment #3:

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

(file #27427)

Julien Bect <jbect>
Sat Feb 9 13:00:53 2013, 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 Feb 9 10:28:17 2013, 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 Feb 9 10:25:21 2013, 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 File(s):

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


Depends on the following items: None found

Items that depend on this one: None found


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

    Date Changed By Updated Field Previous Value => Replaced By
    Sat Feb 23 12:34:05 2013jbectAttached File-=>Added jbFixTestClassesModuleDotMk.patch, #27510
    Wed Feb 20 20:43:43 2013jordighStatusNone=>Fixed
    Sun Feb 10 10:33:46 2013jbectAttached File-=>Added jbFixPrecedenceBug7.patch, #27437
    Sat Feb 9 17:48:22 2013jbectAttached File-=>Added jbFixPrecedenceBug5.patch, #27427
    Sat Feb 9 13:00:53 2013jbectAttached File-=>Added jbFixPrecedenceBug4.patch, #27426
    Sat Feb 9 10:25:21 2013jbectAttached File-=>Added showPrecedenceBugs.zip, #27425

    Back to the top

    Powered by Savane 3.1-cleanup1