On Wed, Jun 24, 2009 at 11:13 PM, Dmitri Gribenko<griboz...@gmail.com> wrote:
> In this light, my proposed solution is just a hack.  For a proper
> solution, I think it is needed to split qt.cpp in two parts:
> * libplpllotqtd.so should contain only classes and functions user
> needs to see (plsetqtdev()/plfreeqtdev());
> * qt.so should contain only functions that are callbacks for plplot
> core.  qt.so will need to be linked against libplplotqtd.so.

Well, here's a patch.  It works for me, please test on your systems.
It is not meant to be committed into svn because:
* it doesn't split qt.cpp into two files, it just adds a preprocessor macro;
* adding yet another one preprocessor macro to qt.cpp makes it
completely unreadable;
* ENABLE_DYNDRIVERS=OFF doesn't work;
* this is the first time I'm working with cmake, so I couldn't find
how to add a different file, so I added a macro...

If it was done the right way (splitting qt.cpp), it would work
essentially the same way.

If the patch is OK, then we need to decide where to put two new source
files, how to name them and handle them properly in cmake (and don't
break ENABLE_DYNDRIVERS=OFF along the way).

Dmitri

-- 
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <griboz...@gmail.com>*/

Attachment: split-qt-cpp.patch.gz
Description: GNU Zip compressed data

------------------------------------------------------------------------------
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to