On Sat, Dec 29, 2018 at 11:20 AM Alan W. Irwin <alan.w.irwin1...@gmail.com> wrote:
> On 2018-12-29 09:17-0000 António Rodrigues Tomé wrote: > > > On Sat, Dec 29, 2018 at 2:28 AM Alan W. Irwin < > alan.w.irwin1...@gmail.com> > > > I'm puzzled my changes do not in any way affect the extqt case. the > ext-qt > > also should never call > > closeQtapp() but in fact it calls it but it is a flaw in code that does > > do not arm because as you said appCounter becomes -1 and nothing is done > > besides the mutex instruction that i'm still uneasy with them, otherwise > it > > would kill the program. > > Actually, this is the source of my unease with this code as well. > Therefore, I plan to drop that closeQtapp call from > plD_tidy_extqt not only to make the code cleaner but also because the > current code would fail if some external qt application tried to use > the combination of extqt with, say, pngqt (e.g., to make a permanent > PNG file corresponding to what was displayed on the screen using > extqt). > just put the mutex instruction in plD_tidy_extqt; The original author never considered the need of any of the others qt drivers from a qt application what for me is amazing... cheers > So unless you have some second thoughts about removal of the > closeQtapp call, I plan to push that change later today after > your current appCounter fix is pushed. > > Alan > __________________________ > Alan W. Irwin > > 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 > __________________________ > -- António Rodrigues Tomé Universidade da Beira Interior Instituto D. Luís (lab associado) email address: art...@gmail.com art...@ubi.pt http://www.researcherid.com/rid/A-5681-2013
_______________________________________________ Plplot-devel mailing list Plplot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/plplot-devel
