Plot property: markerface vs. markerfacecolor

[John W. Eaton]

On 31-Oct-2007, Matthias Brennwald wrote:

| Dear all
| 
| I don't understand how the fill color of a marker in a plot is handled
| in Octave 2.9.15 with gnuplot. Consider the following example:
| 
| ------------------------------------
| h = plot(rand(1,5),'o');
| get(h)
| 
| ans =
| {
| ... (lots of stuff here)
|   marker = o
|   markeredgecolor = auto
|   markerface = none
|   markersize =  1
| ... (more stuff)
| }
| ------------------------------------
| 
| Then:
| ------------------------------------
| get(h,'markerface')
| warning: get: invalid property `markerface'
| ------------------------------------
| 
| This warning message is obviously wrong, as the previous get(...)
| command returned markerface = none. Then, from what I remember using
| Matlab, I tried the following:
| 
| ------------------------------------
| get(h,'markerfacecolor')
| ans = none
| ------------------------------------
| 
| This is what I would get from Matlab (I believe), but it is somehow
| unexpected given that get(h) returns no markerfacecolor property.
| 
| Is this a bug in Octave 2.9.15 or is just me?

Please try the following patch.

Please also report bugs on the address@hidden list.

Thanks,

jwe

src/ChangeLog:

2007-10-31  John W. Eaton  <address@hidden>

        * graphics.cc (line::properties::get): Fix property name
        (markerface -> markerfacecolor).


Index: src/graphics.cc
===================================================================
RCS file: /cvs/octave/src/graphics.cc,v
retrieving revision 1.44
diff -u -u -r1.44 graphics.cc
--- src/graphics.cc     25 Oct 2007 20:41:16 -0000      1.44
+++ src/graphics.cc     31 Oct 2007 16:40:21 -0000
@@ -1825,7 +1825,7 @@
   m.assign ("linewidth", linewidth);
   m.assign ("marker", marker);
   m.assign ("markeredgecolor", markeredgecolor);
-  m.assign ("markerface", markerfacecolor);
+  m.assign ("markerfacecolor", markerfacecolor);
   m.assign ("markersize", markersize);
   m.assign ("keylabel", keylabel);