Thanks, using getGenerator() is much nicer and works. I had just looked at some of the SeriesAttribute subclasses to get a feel for what to do, and I had tried to copy their direct member calls (which had worked prior to Mason 16). Thanks a lot for the tip.


On Wed, Oct 5, 2011 at 8:31 PM, Sean Luke <[log in to unmask]> wrote:
On Oct 5, 2011, at 1:57 PM, Jon Goldstein wrote:

> Do you mean MASON 16? This is the most recent version on the Website.

Yeah, I meant 16.  :-)

> Also, the only problem I had upgrading has to do with the switch from protected to package protected for the generator member variable in SeriesAttribute. In general, if someone builds a new chart generator type, the SeriesAttribute that goes along with it might need access to the corresponding generator to make changes based on user inputs (e.g., look at your implementation of HistogramSeriesAttribute) so it seems that the generator ought to be protected (otherwise, I would argue it ought to be private, not package protected). Anyway, this change broke my code somewhat so I thought would alert you to it and caused me to write a little hack to get it around it (store my own member variable and then short-circuit the buildAttributes() and rebuildGraphicsDefinitions() call in the super class constructor until I call them later in my constructor after storing the other generator variable...)

Jon, can you tell me why SeriesAttributes.getGenerator() wouldn't work?  Do you need to set the generator for some reason?