bugGNU Octave - Bugs: bug #43814, Octave:num-to-str warning is never...

 
 

bug #43814: Octave:num-to-str warning is never issued

Submitted by:  Rik <rik5>
Submitted on:  Sun 14 Dec 2014 12:43:43 AM UTC  
 
Category: InterpreterSeverity: 2 - Minor
Priority: 5 - NormalItem Group: Incorrect Result
Status: FixedAssigned to: None
Originator Name: Open/Closed: Closed
Release: otherOperating System: GNU/Linux

Add a New Comment(Rich Markup)
   

You are not logged in

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

 

Tue 16 Dec 2014 06:17:58 AM UTC, comment #4:

I committed the changeset to stable, since this is a regression from before 3.2.4, and pulled the changes through gui-release and on to the default branch.

Closing report.

Rik <rik5>
Project Administrator
Mon 15 Dec 2014 10:30:40 PM UTC, comment #3:

Rik, your change looks reasonable to me.

John W. Eaton <jwe>
Project Administrator
Mon 15 Dec 2014 05:17:12 AM UTC, comment #2:

I've attached a patch that restores the Octave:num-to-str warning.

I've asked on the Octave Maintainer's list about octave_char_matrix which was supposed to issue this warning, but may be deprecated.

(file #32668)

Rik <rik5>
Project Administrator
Sun 14 Dec 2014 08:53:53 AM UTC, comment #1:

I get a warning if I enable "Octave:mixed-string-concat"

This comes from pt-mat.cc:678:maybe_warn_string_concat
because 111 is here already treated as single quoted string. I think this is strongly related to bug #43815 and #42651

Andreas Weber <andy1978>
Project Member
Sun 14 Dec 2014 12:43:43 AM UTC, original submission:

According to warning_ids(), the following code should produce a warning.

However, no warning is issued and this seems to be a long-standing problem. I checked in versions 3.2.4, 3.4.3, 3.6.4, 3.8.2, and the development branch and nothing happens.

Rik <rik5>
Project Administrator

 

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

Attach File(s):
   
   
Comment:
   

Attached Files
file #32668:  num2str.cset added by rik5 (3KiB - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by jwe (Posted a comment)
  • -unavailable- added by rik5
  • -unavailable- added by andy1978 (Posted a comment)
  • -unavailable- added by rik5 (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 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Tue 16 Dec 2014 06:17:58 AM UTCrik5StatusPatch Submitted=>Fixed
      Open/ClosedOpen=>Closed
    Mon 15 Dec 2014 05:17:12 AM UTCrik5Attached File-=>Added num2str.cset, #32668
      StatusNone=>Patch Submitted
    Mon 15 Dec 2014 05:07:25 AM UTCrik5Carbon-Copy-=>Added -unavailable-

    Back to the top


    Powered by Savane 3.1-cleanup1