On 2013-12-11 21:40-0000 Andrew Ross wrote: > On Wednesday 11 Dec 2013 11:09:15 Andrew Ross wrote: > >> Yes, I've been using this option. Very handy. I've now got the test_diff_psc >> resuilts valgrind clean for C and C++ with standard optimisation. I've not >> (yet) tracked down the colorbar problem with -O3. > > I've traced the problem back to the remove_characters function in > src/pllegend.c . I'm not sure exactly why, but I think the optimisation of the > memmove call is causing problems. I've implemented an alternative algorithm > which seems to cure the valgrind issue and hasn't changed the output.
Hi Andrew: I was able to follow all the neat things you do with pointer arithmetic to implement the new remove_characters; I am impressed with the compactness of your implementation; and I am pleased it cleans up the memory management issue (which was a big concern of mine) that occurred for the optimized version of the old recursive implementation. So my thanks for this effort. 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 __________________________ ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don't have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk _______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
