Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Phil Rosenberg
>> Note that I have tested this up to the point of checking in the >> debugger that the correct fill rule is recognised by the viewer and >> the correct parameter is passed in to the wxWidgets fill function. >> This definitely works correctly both with and without -eofill >> specified for example 2

[Plplot-devel] Helping you to collect evidence to support a fill-rule bug report for wxWidgets on Windows

2017-10-01 Thread Alan W. Irwin
On 2017-10-01 09:49+0100 Phil Rosenberg wrote: Note that I have tested this up to the point of checking in the debugger that the correct fill rule is recognised by the viewer and the correct parameter is passed in to the wxWidgets fill function. This definitely works correctly both with and with

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Alan W. Irwin
On 2017-09-30 19:22-0700 Alan W. Irwin wrote: Anyhow, thanks very much for this fix, and I have changed the status of https://sourceforge.net/p/plplot/bugs/174/ to closed-fixed. I have just discovered a really strange problem with your recent -eofill fix (commit b603fd22). The issue is that va

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Alan W. Irwin
On 2017-10-01 09:49+0100 Phil Rosenberg wrote: [Alan said] With regard to your remark concerning writing a plsfillrule() function and systematically using it throughout src/plargs.c, I wouldn't want to do that myself, but if you or Jim want to make such a change and it passes comprehensive testi

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Phil Rosenberg
Fixed My test for initialisation was incorrect. On 1 October 2017 at 20:50, Alan W. Irwin wrote: > On 2017-09-30 19:22-0700 Alan W. Irwin wrote: > >> Anyhow, thanks very much for this fix, and I have changed the status >> of https://sourceforge.net/p/plplot/bugs/174/ to closed-fixed. > > > I hav

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Phil Rosenberg
On 1 October 2017 at 21:01, Alan W. Irwin wrote: > On 2017-10-01 09:49+0100 Phil Rosenberg wrote: > > [Alan said] >>> >>> With regard to your remark concerning writing a plsfillrule() function >>> and systematically using it throughout src/plargs.c, I wouldn't want >>> to do that myself, but if yo

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Alan W. Irwin
On 2017-10-02 00:14+0100 Phil Rosenberg wrote: On 1 October 2017 at 21:01, Alan W. Irwin wrote: On 2017-10-01 09:49+0100 Phil Rosenberg wrote: [Alan said] With regard to your remark concerning writing a plsfillrule() function and systematically using it throughout src/plargs.c, I wouldn't w

Re: [Plplot-devel] Helping you to collect evidence to support a fill-rule bug report for wxWidgets on Windows

2017-10-01 Thread Alan W. Irwin
On 2017-10-02 00:19+0100 Phil Rosenberg wrote: Ah, okay, so a single "interior loop" should fill with the winding rule, but not with the odd-even rule. That's what I needed to know. I can generate a simple wxWidgets chunk of code totally independent of PLplot to show that this doesn't work. Th

Re: [Plplot-devel] -eofill issues with -dev wxwidgets and possible rewrite of streams and src/plargs.c

2017-10-01 Thread Jim Dishaw
> On Sep 30, 2017, at 2:35 PM, Alan W. Irwin wrote: > > On 2017-09-30 15:06+0100 Phil Rosenberg wrote: > >> Hi Alan >> >> >> On 30 September 2017 at 07:31, Alan W. Irwin >> wrote: >>> On 2017-09-30 02:27+0100 p.d.rosenb...@gmail.com wrote: >>> I suspect this will be a bug in the rende

[Plplot-devel] Is it time to remove some IPC cruft from our wxwidgets-related code?

2017-10-01 Thread Alan W. Irwin
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 w