bugGNU Octave - Bugs: bug #47330, mat2cell error message suffix...


bug #47330: mat2cell error message suffix doesn't match number

Submitted by:  Dan Sebald <sebald>
Submitted on:  Thu 03 Mar 2016 07:11:43 PM UTC  
Category: Octave FunctionSeverity: 3 - Normal
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: devOperating System: Any

Add a New Comment(Rich 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 05 Mar 2016 12:00:47 AM UTC, comment #6:

Thanks, another savannah bug :)

Mike Miller <mtmiller>
Project Administrator
Fri 04 Mar 2016 11:41:28 PM UTC, comment #5:

Mike, did you mean to revert the status and open/closed fields, or was it just that your comment crossed paths with Rik's?

Dan, I agree that this is tiny compared with how "heavy" Octave is in general, but every bit counts.

Lachlan Andrew <lachlan>
Project Member
Fri 04 Mar 2016 10:37:11 PM UTC, comment #4:

Yes, much better if we care about i18n someday.

Mike Miller <mtmiller>
Project Administrator
Fri 04 Mar 2016 10:32:02 PM UTC, comment #3:

I think the lightweight approach is sufficient. I applied Lachlan's patch here (http://hg.savannah.gnu.org/hgweb/octave/rev/8017ce5b2a6e).

Marking as fixed and closing report.

Rik <rik5>
Project Administrator
Fri 04 Mar 2016 08:43:11 PM UTC, comment #2:

I thought of doing the more lightweight approach, but then I thought, What about Octave is lightweight? Dozens of libraries, macro packages, etc., maybe I could make the suffix fairly sophisticated... It's up to you.

Dan Sebald <sebald>
Fri 04 Mar 2016 11:26:26 AM UTC, comment #1:

Well spotted, Dan.

A lighter-weight change is just to write "dimension 1", "dimension 2" etc., as per the attached patch. What do you think?

(file #36546)

Lachlan Andrew <lachlan>
Project Member
Thu 03 Mar 2016 07:11:43 PM UTC, original submission:

The mat2cell mismatch error message prints out a "th" for any dimension such as 1-th, 2-th, 3-th, 4-th.

The attached patch will fix this. For example:

Dan Sebald <sebald>


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

Attach File(s):

Attached Files
file #36546:  bug_47330_mat2cell.cset added by lachlan (903B - text/x-diff)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by mtmiller (Posted a comment)
  • -unavailable- added by rik5 (Posted a comment)
  • -unavailable- added by lachlan (Updated the item)
  • -unavailable- added by sebald (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 9 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 05 Mar 2016 12:00:47 AM UTCmtmillerStatusPatch Submitted=>Fixed
    Fri 04 Mar 2016 10:37:11 PM UTCmtmillerStatusFixed=>Patch Submitted
    Fri 04 Mar 2016 10:32:02 PM UTCrik5StatusPatch Submitted=>Fixed
    Fri 04 Mar 2016 11:26:26 AM UTClachlanAttached File-=>Added bug_47330_mat2cell.cset, #36546
      StatusNone=>Patch Submitted
    Thu 03 Mar 2016 07:11:43 PM UTCsebaldAttached File-=>Added octave-mat2cell_suffix_mismatch-djs_2016mar03.patch, #36537

    Back to the top

    Powered by Savane 3.1-cleanup1