Hi Alan
I haven't really en through that code, but yes I have been using t and yes I'm
happy for that cleanup to occur. The joy of git is that the code history is
still here if needed.
One question i did have though – I remember my code being set up like a ring
buffer, and you saying yours wasn't. But I presume it can still cope with large
transfers bigger than the shared memory block?
So yes, feel free to clean up as you wish.
Phil
Sent from my Windows 10 phone
From: Alan W. Irwin
Sent: 02 October 2017 06:25
To: Phil Rosenberg; PLplot development list
Subject: Is it time to remove some IPC cruft from our wxwidgets-related code?
Hi Phil:
Thanks for all the testing and bug-fixing for wxwidgets that you have
being charging through this weekend.
During the course of that testing (presumably on Windows) I am going
to assume you just used the default given by
option(PL_WXWIDGETS_IPC3 "Use the three-semaphores approach for wxwidgets IPC"
ON)
(i.e., you did not specifically use -DPL_WXWIDGETS_IPC3=OFF). That
default case corresponding to -DPL_WXWIDGETS_IPC3=ON is what I
constantly use on Linux for wxwidgets, and I am happy with it.
Are you happy with your recent experience with the three-semaphores
IPC approach on Windows, i.e., has everything worked as well as with the old
IPC approach with no noticeable slowdowns? If so and whenever you give
the OK, I propose to remove from our wxwidgets-related code your
original IPC approach which involved a circular buffer code and a
mutex, i.e., all code which is currently compiled only if the
PL_WXWIDGETS_IPC3 macro is NOT #defined. That change should make both
the -dev wxwidgets code and wxPLViewer code much easier to understand
which is why I am pushing for this change.
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
__________________________
------------------------------------------------------------------------------
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
