Hi Ulrich, On Wed, Feb 10, 2010 at 8:13 AM, Ulrich Hertlein <[email protected]> wrote: > On 10/02/10 9:27 , Paul Martz wrote: >> #define NOTIFY(level) if (isNotifyEnabled(level)) osg::notify(level) > > Can we please prefix that to something like 'OSG_NOTIFY'? Please?!? > > Macros are one of these nasty features that happily trample all over other > peoples code > and don't obey namespaces (or anything really). So let's say somebody is > naving a NOTIFY > in an enum he's in for a nasty compile error... :-(
I was hoping to avoid being long winded with the macro name, but you do make a good point about the potential issue of gobbling up names that might be used in client apps and 3rd part libs. I've converted all the NOTIFY usage across to OSG_NOTIFY and checked it into svn/trunk. Cheers, Robert. _______________________________________________ osg-users mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-users-openscenegraph.org
