Re: changing genprop.awk to handle radio_values

[John W. Eaton]

On 14-Sep-2007, Shai Ayal wrote:

| the attached patch will emmit the following set function if the "a"
| modifier is used:
| 
| void set_NAME (const octave_value& val)
|   {
|     TYPE  tmpNAME (NAME);
|     tmpNAME = val;
|     set_NAME (tmpNAME);
|   };
| 
| this will allow the error check in set_NAME (const TYPE& NAME) to be used
| 
| 
| 2007-09-14 Shai Ayal <address@hidden>
| 
|     * genprop.awk: added the "a" modifier which creates an assignment
| set_NAME(const octave_value& val) function

I applied this patch with a few minor changes.  First, I just used
"tmp" instead of tmpNAME, and I also added some more info to the
description in the comments.  Is the following a correct and
reasonable statement?

  ##   a:  The octave_value version of the set function will use assignment:
  ##
  ##         void
  ##         set_NAME (const octave_value& val)
  ##         {
  ##           TYPE tmp (NAME);
  ##           tmp = val;
  ##           set_NAME (tmp);
  ##         }
  ##
  ##       This is useful for things like the radio_value classes which
  ##       use an overloaded assignment operator of the form
  ##
  ##         radio_property& operator = (const octave_value& val);
  ##
  ##       that preserves the list of possible values, which is different
  ##       from what would happen if we simply used the
  ##
  ##         TYPE (const octave_value&)
  ##
  ##       constructor, which creates a new radio_property and so cannot
  ##       preserve the old list of possible values.

Which properties should be tagged with "a" in graphics.h.in?

Thanks,

jwe