bugGNU Octave - Bugs: bug #51899, openvar() function should validate...

 
 

bug #51899: openvar() function should validate its inputs

Submitter:  Rik <rik5>
Submitted:  Wed 30 Aug 2017 09:03:26 PM UTC
   
 
Category:  Octave Function Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Other
Status:  Fixed Assigned to:  None
Originator Name:  Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Wed 30 Aug 2017 11:06:38 PM UTC, comment #2: 

I added both bits of input validation in this cset http://hg.savannah.gnu.org/hgweb/octave/rev/529272d1c271.  Fixed, closing report.

Rik <rik5>
Group administrator
Wed 30 Aug 2017 09:06:03 PM UTC, comment #1: 

It might also be nice to issue a warning if the octave_link is not active, when running octave-cli or octave --no-gui.

Mike Miller <mtmiller>
Group Member
Wed 30 Aug 2017 09:03:26 PM UTC, original submission:  

I was trying the new Variable Editor and I used


y = 1:10
openvar (y)


There were two warning messages issued immediately in the Command Window.


warning: implicit conversion from matrix to sq_string
warning: implicit conversion from range to sq_string


In addition, the Variable Editor did pop open a tab with a crazy string for the variable name which it said it could not find.

The correct syntax should have been


openvar ("y")


where "y" is a string.  Instead, I got something like


openvar (char (y))


I understand how this happened, but it would be nicer if the openvar function would validate its inputs just a bit.

Rik <rik5>
Group administrator

 

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

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-30 rik5 StatusConfirmed Fixed
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code