Hi Alan I looked at this last night and found two issues. The first was a problem in plstrl, where the buffer was not turned off before the call to c_plmtex, meaning an extra text command ended up in the buffer to be written to location 0,0. I have fixed this and am about to commit it.
The second problem is that when using wxPLViewer the driver doesn't actually have a device to render to, so cannot find the size. In fact the driver call just returns without doing anything meaning that the size doesn't get set and is just left as zero. The result is that in x26 the legend key and text are overwritten. A fix for this is possible but not trivial so for now pls->has_string_length will only be set to TRUE if the driver has a valid wxDC to draw to. For the viewer case the plplot derived size must be used instead. I'm about to commit these changes now. Phil On 28 February 2015 at 09:42, Phil Rosenberg <p.d.rosenb...@gmail.com> wrote: > Thanks Alan, I will look at the patch and see what I can do. > > Phil > ________________________________ > From: Alan W. Irwin > Sent: 28/02/2015 00:25 > To: Phil Rosenberg > Cc: PLplot development list > Subject: RE: [Plplot-devel] wxWidget driver bug reporting > > On 2015-02-27 22:12-0000 Phil Rosenberg wrote: > >> Hi Alan > >> Yes I believe this is doable in wxWidgets. I'm sure I remember > coding it up when I added newline support so I think the code is > already in the driver if you look. I guess I never realised this > needed plumbing into the core code. > >> Should be a very easy fix if you know how to feed the sizes back in. > > I have put in that necessary plumbing (see attached patch) but it > isn't working correctly (produces strange looking results for example > 26). > > Note, for git format-patch I used the --ignore-all-space option so > that the patch does not reflect the local styling changes I made, but > does give the essence of most of the changes you should need to get > the example 26 legend size to be correct. > > I hope some minor correction is all that is needed to make this patch > work, but I cannot figure out what that correction is (probably > because I am not that familiar with the wxwidgets device code) so I > hope you can figure out that correction instead. > > 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 > __________________________ ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
