Hi Alan

On 31 July 2018 at 19:21, Alan W. Irwin <alan.w.irwin1...@gmail.com> wrote:
> Hi Phil:
>
> I am moving this discussion to plplot-devel where it belongs.

Sorry, I hadn't realised we'd gotten off list. Thanks for moving it back.

>
> I think your backwards-incompatible changes [h,l,s ==> c1,c2,c3
> changed member names] in the PLControlPt struct are likely a good idea
> since h,l,s was a misnomer.  Nevertheless, those changes did cause
> important build problems in the Tk-related part of the build which I
> have now fixed (commit 94e2e5b01).

Is this struct really part of our public API? It is not mentioned
anywhere in the docs as far as I can tell so I presumed this was an
internal use only structure.

>
> Example 12 (which uses RGB interpolation) shows the effect of your
> change.  The old code produced
> <http://plplot.org/examples-data/demo12/x12.01.png> which shows fairly
> discontinous colour results from 1982 to 1986, while the new code
> produces much smoother colour changes in that date range.  So from
> this evidence I feel your change is a nice win.
>
> I previously said (in response to your question about how you
> should edit README.release):
>
>> I assume you will want to insert a paragraph concerning
>
> your change in a new section 2.8 of "Improvements relative to the
> previous release".  Also, since you changed the semantics of the two
> functions (although not their API) for the RGB case you should
> probably specifically warn the users of this forthcoming release about
> that semantics change by adding a one-sentence new section 1.10 in
> "OFFICIAL NOTICES FOR USERS" that refers to your new section 2.8 for
> the details.
>
> Since your backwards-incompatible API change to the PLControlPt struct is
> going
> to cause build problems (just like it did in our Tk-related code)
> for the minority of our users who actually use that struct
> in their code, you should expand that suggested single sentence in 1.10 to
> warn users of this specific change to PLControlPt.
>
> I look forward to your commit with the suggested changes in README.release.

They'll arrive this afternoon all being well.

>
>
> Alan
> __________________________
> Alan W. Irwin
>
> Programming affiliations with the FreeEOS equation-of-state
> implementation for stellar interiors (freeeos.sf.net); the Time
> Ephemerides project (timeephem.sf.net); PLplot scientific plotting
> software package (plplot.sf.net); the libLASi project
> (unifont.org/lasi); the Loads of Linux Links project (loll.sf.net);
> and the Linux Brochure Project (lbproject.sf.net).
> __________________________
>
> Linux-powered Science
> __________________________

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to