Agreed. Changing the variable names now is relatively straightforward and 
it is much better to have a clear API.

Andrew

On Sun, Nov 09, 2014 at 09:39:37AM -0800, Alan Irwin wrote:
> On 2014-11-09 11:08-0000 phil rosenberg wrote:
> 
> > HiI am just writing the documentation for the new map API. I wonder if some 
> > of the variable names are now misleading. For example type can either be 
> > one of the plplot built in map types or the filename of a shapefile, so 
> > perhaps name or file might be better. Perhaps more importantly in a 
> > shapefile the coordinates may not be lat/lon (this is true for the ones 
> > used in the new example) so the variables minlong etc are misleading, they 
> > should perhaps be minx etc instead.
> > I realize however that people have been implementing the new functions in 
> > other bindings so this would have been better flagged a few weeks ago, so 
> > I'm sorry if this generates extra unnecessary work. Should we change them 
> > or leave them the same?
> 
> Hi Phil:
> 
> I am all for explanatory argument names.  So please go ahead and
> choose what you think are the best plmap* argument names for C, C++,
> and the DocBook documentation.  I will take responsibility for making
> the corresponding changes in the 4 languages I have dealt with.
> 
> Alan
> 
> __________________________
> Alan W. Irwin
> 
> Astronomical research affiliation with Department of Physics and Astronomy,
> University of Victoria (astrowww.phys.uvic.ca).
> 
> 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
> __________________________
> 
> ------------------------------------------------------------------------------
> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel

------------------------------------------------------------------------------
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to