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

 
 

bug #50163: set() is case-sensitive

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

Add a New Comment (Rich Markup)
   

You are not logged in

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

 

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>
Project 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>
Project Member
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)
  •  

    Do you think this task is very important?
    If so, you can add your encouragement to it.
    This task has 0 encouragements so far.

    Only project 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.5