Re: setting a default for non-existent property

[Ben Abbott]

On Thursday, May 07, 2009, at 03:15PM, "John W. Eaton" <address@hidden> wrote:
>On  7-May-2009, Ben Abbott wrote:
>
>| I accidently set a default for a non-existent property and discovered  
>| I had to quit octave to recover.
>| 
>| For example ...
>| 
>| octave:5> set (0, "defaultfigure_bad_property", 0)
>| octave:6> figure
>| error: set: unknown property "_bad_property"
>| error: error setting default property _bad_property
>| error: __go_figure__: failed to create figure handle
>| error: called from:
>| error:   /Users/bpabbott/Development/mercurial/local_clone/scripts/ 
>| plot/figure.m at line 67, column 9
>| 
>| I assume the same is true for all defaults (text, axes, figure, etc).
>| 
>| Can someone with better c/c++ skills take a look at fixing this?
>
>I checked in the following change:
>
>  http://hg.savannah.gnu.org/hgweb/octave/rev/1e5c11890f85
>
>It seems to work for me for your example above.  Please try it out and
>let me know whether it causes any problems.
>
>Thanks,
>
>jwe

I pulled, and went through a full autogen/configure/make

changeset:   9185:1e5c11890f85
tag:         tip
user:        John W. Eaton <address@hidden>
date:        Thu May 07 15:14:19 2009 -0400
summary:     check for invalid property names when setting defaults

octave:3> close all
octave:4> figure(1)
octave:5> plot(1:10)
error: set: unknown property "horizontalalignment"
error: set: unknown property "horizontalalignment"
error: set: unknown property "horizontalalignment"
error: set: unknown property "horizontalalignment"
error: set: unknown property "verticalalignment"
error: set: unknown property "verticalalignment"
error: set: unknown property "verticalalignment"
error: set: unknown property "rotation"
error: error setting default property fontname
error: __goaxes__: unable to create graphics handle
error: called from:
error:   /Users/bpabbott/Development/mercurial/local_clone/scripts/plot/axes.m 
at line 40, column 9
error:   /Users/bpabbott/Development/mercurial/local_clone/scripts/plot/gca.m 
at line 45, column 9
error:   
/Users/bpabbott/Development/mercurial/local_clone/scripts/plot/__plt_get_axis_arg__.m
 at line 63, column 4
error:   /Users/bpabbott/Development/mercurial/local_clone/scripts/plot/plot.m 
at line 178, column 18

I assume you're able to plot, so I'll take another shot at building.

However, the "__goaxes__" error should be fixied, but I can't find it.

Ben