Re: [Plplot-devel] PL_MAX_OPT_TABLES PL_NSTREAMS wxWidgets Driver

2015-02-14 Thread laurent Berger
I think that my previous thread was wrong. Finally I'm not sure that my code uses in a rigth way plplot( sorry for my bad english) In my program I open many windows with images. If I open 10 windows there is no problem but with 11 windows my program crash For each image I open a panel which incl

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 08:27- Phil Rosenberg wrote: > Hi Alan and Jim > > I tried applying the patches on my system and they failed too. In fact > my patch wouldn't apply at all, even on a clean branch. It had > whitespace errors and a commit that changed the case of a filename, > which confused git on

Re: [Plplot-devel] PL_MAX_OPT_TABLES PL_NSTREAMS wxWidgets Driver

2015-02-14 Thread Phil Rosenberg
Hello Laurent Yes I agree that if such a limit does exist on the number of plplot windows which you have in an application then this needs addressing. I will try to look at this as soon as I can. You could speed up the process by sending a minimal example which displays the problem. From your des

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Phil Rosenberg
Hi Alan I think our emails crossed - or at least I didn't see yours before I sent mine. However I think what I said in my last email still stands. If we all use the merged patch as a basis for future work on this topic then we can all be happy, and when I commit some or all of this patch to master

Re: [Plplot-devel] PL_MAX_OPT_TABLES PL_NSTREAMS wxWidgets Driver

2015-02-14 Thread laurent Berger
Hi phil, Sorry I make a mistake about PL_MAX_OPT_TABLES there is no problem it was in my code. About PL_NSTREAMS limits it's a problem. you can reproduce problem with PL_NSTREAMS changing some lines codes in wxPlplotDemo.cpp : original : MyFrame *frame = new MyFrame( _T( "wxPLplot demo" ) ); fra

Re: [Plplot-devel] PL_MAX_OPT_TABLES PL_NSTREAMS wxWidgets Driver

2015-02-14 Thread laurent Berger
Finally I think that's may be interesting for you the other bug in my code. In wxPlplotDemo.cpp you have to replace constructor with this new constructor ond plplot crash : //! Constructor of our custom frame, where the Menu is created and a // a wxPLplotwindow is inserted into the frame. We plo

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 14:52- Phil Rosenberg wrote: > Hi Alan > I think our emails crossed - or at least I didn't see yours before I > sent mine. However I think what I said in my last email still stands. > If we all use the merged patch as a basis for future work on this > topic then we can all be hap

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 10:16-0800 Alan W. Irwin wrote: > On 2015-02-14 14:52- Phil Rosenberg wrote: > >> Hi Alan >> I think our emails crossed - or at least I didn't see yours before I >> sent mine. However I think what I said in my last email still stands. >> If we all use the merged patch as a basis

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Phil Rosenberg
I found an hour to go through all the examples this afternoon and found a number of items that need fixing. The most important of which was that fills weren't working. My list of things to fix follows, along with my guess as to whether it is a wxWidgets driver issue or a buffer issue - this is jus

Re: [Plplot-devel] PL_MAX_OPT_TABLES PL_NSTREAMS wxWidgets Driver

2015-02-14 Thread Phil Rosenberg
Thanks Laurent, I will look into it asap Phil On 14 February 2015 at 16:44, laurent Berger wrote: > Finally I think that's may be interesting for you the other bug in my code. > In wxPlplotDemo.cpp you have to replace constructor with this new > constructor ond plplot crash : > > //! Constructo

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Jim Dishaw
> On Feb 14, 2015, at 3:54 PM, Phil Rosenberg wrote: > > I found an hour to go through all the examples this afternoon and > found a number of items that need fixing. The most important of which > was that fills weren't working. > My list of things to fix follows, along with my guess as to wh

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
> The attached patch should include all our commits and the conflict > fix. Jim and Alan and anyone else interested in testing this can you > please do the following steps > git checkout master > git fetch > git merge origin master --ff-only > git branch wxAndBufTestMaster > git checkout wxAndBufTe

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 20:54- Phil Rosenberg wrote: > I found an hour to go through all the examples this afternoon and > found a number of items that need fixing. The most important of which > was that fills weren't working. > My list of things to fix follows, along with my guess as to whether it > is

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 16:10-0500 Jim Dishaw wrote: > Sure. First I need to figure out if I should rebase my branch to yours or if I should use your branch and patch. Any suggestions? It hasn't happened yet, but I have given Phil the strong encouragement to push his integrated patch series (which contains

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Phil Rosenberg
Done - I've pushed the changes So Jim as Alan said, feel free to pull from the repo. I guess you should then be able to rebase your working branch onto that. I will work to fix the bugs over the next week or so. > then the highest priority for me would be the page navigation fix Here you have a

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 13:44-0800 Alan W. Irwin wrote: > Once that push happens, you should simply create a new topic branch > from the master tip, and then make your further changes beyond what > you sent us already. @Jim: If you have not subscribed already to the git feed then you should do so now (by l

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
Hi Phil: On 2015-02-14 13:28-0800 Alan W. Irwin wrote: > IMPORTANT... I strongly encourage you to go ahead and push > this whole integrated series of commits to master. Glad to see you did this. > * Examples 8, 11, 16, 19 (last page), > 20 (at least first page), 21 (second page), 24, 25, 27 (so

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 22:50- Phil Rosenberg wrote: > Let me know which (pgup/pgdn or enter) you prefer. Replace with enter for now (and add the exit functionality when enter is struck with no pages left) to be consistent with all the rest of our devices and what was done for the old wxwidgets device.

[Plplot-devel] Resize issues

2015-02-14 Thread Alan W. Irwin
To Jim and Phil: I notice that -dev xwin is always one behind whatever resize command you give it. To prove this for yourself, grab the edge of the xwin widget and resize it much larger or smaller. Nothing happens to the size of the plot until the next time you resize it, and if you watch closel

[Plplot-devel] Intrusive style change just made to master branch

2015-02-14 Thread Alan W. Irwin
I hope I chose my timing well for this intrusive style change that I have just pushed so that you guys can easily carry on tomorrow with further work on your plbuf and wxwidgets topic branches that you branch from the (now restyled) master branch. @Phil: One potential issue I noticed as a result o

Re: [Plplot-devel] Redesigned wxWidgets Driver

2015-02-14 Thread Alan W. Irwin
On 2015-02-14 13:28-0800 Alan W. Irwin wrote: > [] I also noticed A "no such file" message > when I attempt to move to beyond the first page of the example 20 > plot, but that now occurs for all devices so I am pretty sure this is > a master branch regression and nothing to do with wxwidgets/