[Octave-bug-tracker] [bug #37290] gnuplot / demo plotyy followed by sombrero errors out

[Dan Sebald]

Follow-up Comment #35, bug #37290 (project octave):

Ben,

That error is what happens after the patch is applied and a "reset (gcf)" is
input.  The way the code is set up, read-only variables are in the factory
defaults:


property_list::pval_map_type
axes::properties::factory_defaults (void)
{
  property_list::pval_map_type m = base_properties::factory_defaults ();

  m["position"] = default_axes_position ();
  m["box"] = "on";
{snip}
  m["tightinset"] = Matrix (1, 4, 0.0);
{snip}
  m["autopos_tag"] = "none";

  return m;
}


but they are not in this if-else "set" structure:


void
axes::properties::set (const caseless_str& pname_arg, const octave_value&
val)
{
  const std::set<std::string>& pnames = all_property_names ();

  caseless_str pname = validate_property_name ("get", go_name, pnames,
pname_arg);

  if (error_state)
    return;

  if (pname.compare ("position"))
    set_position (val);
  else if (pname.compare ("box"))
    set_box (val);
  else if (pname.compare ("colororder"))
{snip: no 'tightinset' in this list}
  else if (pname.compare ("autopos_tag"))
    set_autopos_tag (val);
  else
    base_properties::set (pname, val);
}


The point is that there is no way of knowing which are read-only variables
with this construct, so if one restores the factory settings the only thing is
to know beforehand which read-only variable names to condition out of the
reset.  Not difficult to do, but again it feels like real patchwork for only
marginal gain.

If you want to replicate the issue, try


>> get (gca, 'tightinset')
ans =

   0   0   0   0

>> set (gca, 'tightinset', [0 0 0 0])
error: set: unknown property "tightinset"


Regarding the 'set(gca, "defaultlinewidth", 2.3)', you are correct.  Maybe I
was confused by the axes "linewidth", but as written the "width" is checked in
the "line" object.  I will look at that again.  Thanks.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/bugs/?37290>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/