Hi Alban,

Thanks for this. I've committed it to svn. 

Andrew

On Thu, Aug 20, 2009 at 11:50:34AM +0100, Rochel, Alban wrote:
> Hello all,
> 
> Here is a patch for the Qt driver. It fixes a couple of minor bugs, and it 
> also adds the possibility to pass PLplot command line arguments to the qtext 
> driver (embedded widget in a Qt application) and handles flush and clear for 
> the interactive devices. However, there still are issues with these 
> implementations:
> 
> - Command line arguments for the qtext driver:
> The user has to pass argc and argv to plsetqtdev (the function that 
> associates the widget to the device). But in order to create a QApplication, 
> the user has to pass argc and argv to the QApplication constructor before, 
> which filters out the arguments it recognizes, including the "-bg" argument 
> (conflict between Qt and PLplot). I believe the only way to fix this other 
> than changing "-bg" into something else in PLplot is to make a deep copy of 
> argv before passing it to the QApplication constructor and passing this copy 
> to plsetqtdev. Thus, it's up to the application developer to handle such 
> matters. You can see this issue with qt_example:
> "qt_example -bg FF000" will display a Qt warning message and go on with the 
> default bg colour
> "qt_example -cmap0 _whatever_" will work properly
> 
> - flushing: it works, but example x17c is *slow* (about 30 times slower than 
> xwin on my system!). This is because the Qt driver will replot everything 
> from the last clear() everytime the flush is done. This is linked to the way 
> the driver was designed, and despite my efforts, I haven't found a quick and 
> easy way to fix this. I still plan to fix this, but it will require some 
> redesign. In the meantime, people who wish to use the Qt driver to animate 
> stuff may be happy to use the new "lines_antialiasing" command-line option 
> which allows to (dis/en)able lines antialiasing (enabled by default), which 
> makes the driver about 5 times slower than xwin.
> 
> Despite these flaws, as this update adds features and should not break 
> anything, I believe it's worth adding to the svn trunk.
> 
> Alban

> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
> trial. Simplify your report design, integration and deployment - and focus on 
> what you do best, core application coding. Discover what's new with 
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> _______________________________________________
> Plplot-devel mailing list
> Plplot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/plplot-devel


------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to