[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [changeset] property value not paired with name in __add_datasource_
From: |
John W. Eaton |
Subject: |
Re: [changeset] property value not paired with name in __add_datasource__.m |
Date: |
Tue, 02 Sep 2008 14:52:56 -0400 |
On 1-Sep-2008, David Bateman wrote:
| Ben Abbott wrote:
| >
| > On Aug 31, 2008, at 12:52 PM, Ben Abbott wrote:
| >
| >>
| >> On Aug 31, 2008, at 11:15 AM, John W. Eaton wrote:
| >>
| >>> On 31-Aug-2008, Michael Goffioul wrote:
| >>>
| >>> | The following patch (in graphics mq) solves the problem
| >>> |
| >>> http://hg.tw-math.de/octave-graphics-mq/file/9d2c6e36870f/set_empty_args
| >>>
| >>>
| >>> I applied it.
| >>>
| >>> Thanks,
| >>>
| >>> jwe
| >>
| >> I started fresh, but the default branch still fails to build ... new
| >> error though :-)
| >>
| >> ../../run-octave -f -q -H -p . --eval "geometryimages ('voronoi',
| >> 'eps');"
| >> error: set: invalid number of arguments
| >> error: called from:
| >> error:
| >>
/Users/bpabbott/Development/mercurial/octave-3-0-0/scripts/plot/__plt2vv__.m
| >> at line 83, column 7
| >> error:
| >> /Users/bpabbott/Development/mercurial/octave-3-0-0/scripts/plot/__plt2__.m
| >> at line 68, column 14
| >> error:
| >> /Users/bpabbott/Development/mercurial/octave-3-0-0/scripts/plot/__plt__.m
| >> at line 79, column 10
| >> error:
| >> /Users/bpabbott/Development/mercurial/octave-3-0-0/scripts/plot/plot.m
| >> at line 189, column 5
| >> error:
| >>
/Users/bpabbott/Development/mercurial/octave-3-0-0/doc/interpreter/geometryimages.m
| >> at line 45, column 5
| >>
| >> make[3]: *** [voronoi.eps] Error 1
| >> make[2]: *** [interpreter] Error 2
| >> make[1]: *** [doc] Error 2
| >> make: *** [all] Error 2
| >>
| >> I identified the problem is in __plt2vv__.m
| >>
| >> [...]
| >> 69 hg = hggroup ();
| >> 70 retval = hg;
| >> 71 properties = __add_datasource__ ("__plt2vv__", hg, {"x", "y",
| >> "z"},
| >> 72 properties{:});
| >> 73
| >> 74 h = line (x, y, "keylabel", key, "color", color,
| >> 75 "linestyle", options.linestyle,
| >> 76 "marker", options.marker, "parent", hg);
| >> 77
| >> 78 __add_line_series__ (h, hg);
| >> 79 if (! isempty (properties))
| >> 80 set (hg, properties{:});
| >> 81 endif
| >>
| >> Prior to line 71"properties" is set as
| >>
| >> {
| >> [1,1] = LineWidth
| >> [1,2] = 3
| >> }
| >>
| >> Subsequently, "properties is set as
| >>
| >> {
| >> [1,1] = LineWidth
| >> }
| >>
| >> ... there is no value paired with the property name. I'm not familiar
| >> with the intended behavior, but have attempted a fix. Changeset is
| >> attached. With this in place, I now get warnings but haven't looked
| >> deeper.
| >>
| >> geometryimages ('voronoi', 'eps')
| >> warning: legend: plot data is empty; setting key labels has no effect
| >> warning: legend: ignoring extra labels
| >>
| >> Ben
| >>
| >> <__add_datasource__.txt>
| >
| > I found another instance of "set" being used with empty properties,
| > combined patch attached.
| >
| > Ben
| >
| >
| Working back from my previous patch that introduced this issue and
| taking into account the fixes already made by Michael, here is the
| changeset that gets all of these
I applied it.
Thanks,
jwe
- Re: [changeset] property value not paired with name in __add_datasource__.m, David Bateman, 2008/09/01
- Re: [changeset] property value not paired with name in __add_datasource__.m,
John W. Eaton <=
- Re: [changeset] property value not paired with name in __add_datasource__.m, David Bateman, 2008/09/01
- Re: [changeset] property value not paired with name in __add_datasource__.m, Ben Abbott, 2008/09/01
- Re: [changeset] property value not paired with name in __add_datasource__.m, Ben Abbott, 2008/09/01
- Re: [changeset] property value not paired with name in __add_datasource__.m, David Bateman, 2008/09/01
- Re: [changeset] property value not paired with name in __add_datasource__.m, Ben Abbott, 2008/09/01
- how to verify our sources are sync'd, Ben Abbott, 2008/09/01
- Re: how to verify our sources are sync'd, David Bateman, 2008/09/01
- Re: how to verify our sources are sync'd, John W. Eaton, 2008/09/01