I've committed a fairly invasive patch to cleanup the plplot namespace. 

Now plConfig.h only contains definitions which need to be externally 
visible because they are referenced in plplot.h or because the examples
use them. All other definitions are only in config.h. The distinction 
between these had got blurred, particularly since all cmake builds
have config.h which was not the case with the old build system.

All external definitions start with PL_ to keep the external namespace
clean. 

I've also redefined some internal HAVE_* definitions to PL_HAVE_* to 
fix the long-standing python warnings due to python leaking HAVE_* 
macros.

Finally I've moved some definitions for replacement isnan functions
etc from plplot.h to plplotP.h. It's fine for plplot to use these
internally, but it is not really our role to provide external 
versions of these which could potentially clash with other code.

This is a fairly big change touching lots of different source files, 
so please check that everything still works as you expect.

We could go one stage further and split of those definitions and
macros which are only used in the C / C++ examples into a separate
config file in the examples directory. This would leave plConfig
as an even smaller file.

WARNING: some of these definitions have been externally visible for
a long time. Users should not have been using them, but this 
does not mean that they didn't. You may want to check your user
code.

Andrew

------------------------------------------------------------------------------
Enter the BlackBerry Developer Challenge  
This is your chance to win up to $100,000 in prizes! For a limited time, 
vendors submitting new applications to BlackBerry App World(TM) will have
the opportunity to enter the BlackBerry Developer Challenge. See full prize  
details at: http://p.sf.net/sfu/Challenge
_______________________________________________
Plplot-devel mailing list
Plplot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/plplot-devel

Reply via email to