[Openjade-checkins] CVS: jade/grove Node.cxx,1.6,1.7

[Neil R. <ne...@oc...>]

On May 17, Travis Spencer (tr...@us...) wrote:
 > Update of /cvsroot/openjade/jade/grove
 > In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv21252
 > 
 > Modified Files:
 > 	Node.cxx 
 > Log Message:
 > Expanded the ENUM macro by hand rather than using the preprocessor to
 > paste tokens together to avoid issues that some compilers were having.

Can you back out this change and just change the macro definition from

#define ENUM(e, E) case EntityType::##e: value.set(ComponentName::id##E); break;

to

#define ENUM(e, E) case EntityType::e: value.set(ComponentName::id##E); break;

Seems better than a wholesale manual replacement of the macro.

I maintain the Debian package for OpenJade, and it has this change.  I thought
I had discussed this change on this list in the past, so I'm surprised it is
not checked in.  I'd be happy to work with you to test your changes, or if I
can get access to check in, I'd be happy to merge the Debian changes in
someday.

-- 
Neil Roeth