bugGNU Octave - Bugs: bug #50163, set() is case-sensitive

 
 

bug #50163: set() is case-sensitive

Submitter:  Guillaume <gyom>
Submitted:  Fri 27 Jan 2017 11:09:59 AM UTC
   
 
Category:  Plotting Severity:  3 - Normal
Priority:  5 - Normal Item Group:  Unexpected Error or Warning
Status:  Fixed Assigned to:  None
Originator Name:  Guillaume Open/Closed:  * Closed
Release:  * dev Operating System:  * Any
Fixed Release:  None Planned Release:  None
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 30 Jan 2017 09:50:39 AM UTC, comment #3: 

Many thanks, Markus and Rik!

Guillaume <gyom>
Sun 29 Jan 2017 02:40:02 AM UTC, comment #2: 

I committed the cset here (http://hg.savannah.gnu.org/hgweb/octave/rev/1b2525cdd110).  I changed the call to "::lower" to just "lower" to match other instances in the file.

Also, even though this has a low probability of causing problems, it doesn't really fix something critical so I changed it to apply to the default branch rather than the stable branch.

Rik <rik5>
Group administrator
Fri 27 Jan 2017 06:52:43 PM UTC, comment #1: 

Attached is a patch that converts the property name to lower-case before matching.

(file #39575)

Markus M├╝tzel <mmuetzel>
Group administrator
Fri 27 Jan 2017 11:09:59 AM UTC, original submission:  

It seems that property names are case-sensitive when used in set() with two input arguments:


>> H = figure;
>> get(H, 'resize')
ans = on
>> get(H, 'Resize')
ans = on
>> set(H, 'resize')
[ off | {on} ]
>> set(H, 'Resize')
error: set: unknown property
>> set(H, 'Resize','off')
>> get(H, 'Resize')
ans = off


Guillaume <gyom>

 

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

Attach Files:
   
   
Comment:
   

Attached Files
file #39575:  bug50163_set_case-insensitive.patch added by mmuetzel (909B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by rik5 (Posted a comment)
  • -email is unavailable- added by mmuetzel (Updated the item)
  • -email is unavailable- added by gyom (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 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-01-29 rik5 Open/ClosedOpen Closed
    2017-01-29 rik5 StatusPatch Submitted Fixed
    2017-01-27 mmuetzel Attached File- Added bug50163_set_case-insensitive.patch, #39575
        StatusNone Patch Submitted
        Operating SystemGNU/Linux Any

    Back to the top

    Powered by Savane 3.13-4b48.
    Corresponding source code