I've just committed the plstream:fill changes
What was the issue with the pltr[0-2] functions? Do they need changing too?
Alan - some docbook changes managed to get into my first commit (they were 
staged, but not commited, which I thought was enough to allow me to swap 
branches but I guess not). I didn't spot them until after my second commit. I 
then did a --amend to remove them from the second commit, but not the first. 
Sorry about that. Thinking now, I assumed it isn't possible to amend after a 
further commit, but perhaps it is. Still learning I guess.
Phil
       From: Alan W. Irwin <ir...@beluga.phys.uvic.ca>
 To: Andrew Ross <andrewr...@users.sourceforge.net> 
Cc: phil rosenberg <philip_rosenb...@yahoo.com>; PLplot development list 
<Plplot-devel@lists.sourceforge.net> 
 Sent: Wednesday, 22 October 2014, 10:03
 Subject: Re: [Plplot-devel] Why is the set_stream() call commented out 
forplstream::fill, etc.
   
On 2014-10-22 09:07+0100 Andrew Ross wrote:

> On Wed, Oct 22, 2014 at 07:40:34AM +0000, phil rosenberg wrote:
[...]
>> a) Just use the C plfill option

> I could live with a) I guess despite my
reservations. We have the C++ version for the direct calls, and just
use the C version for the callback.

Andrew, if you are willing to go along with Phil's idea of using C
callbacks, then that is fine with me too.

Thus, it appears we have finally arrived at consensus (including ,as I
brought up before, the same C callback solution for pltr[0-2] for
consistency with the C callback solution for plfill).

Phil, will you do the honors with a commit to that effect?



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
__________________________


  
------------------------------------------------------------------------------
Comprehensive Server Monitoring with Site24x7.
Monitor 10 servers for $9/Month.
Get alerted through email, SMS, voice calls or mobile push notifications.
Take corrective actions from your mobile device.
http://p.sf.net/sfu/Zoho
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to