[ http://issues.apache.org/jira/browse/XERCESC-1327?page=comments#action_57884 ] Robert Buck commented on XERCESC-1327: --------------------------------------
> By the way: "Parsers don't kill people. People do.". ;) Perhaps you are trying to make light of the matter, but I hardly find it funny. This is a serious issue, as are most of the defects I have been finding. Would it be funny if a root cause analysis of the explosion of a prototype JSF fighter, that killed one of our military personel, and cost billions, was traced back to a coding error in Xerces-C, and introduced by yourselves? Yes, you are dead right; people do kill people. The very reason I hound people on these sorts of things is because, yes, the devil _is_ in the details. Where loss of life is a factor, details become paramount. > Change all enum values from ALLCAPS to MixedCase > ------------------------------------------------ > > Key: XERCESC-1327 > URL: http://issues.apache.org/jira/browse/XERCESC-1327 > Project: Xerces-C++ > Type: Bug > Components: Build > Versions: 2.6.0 > Environment: All > Reporter: Robert Buck > > The enum values in Xerces-C are in ALLCAPS, which violates general > programming practice, which reserves ALLCAPS identifiers for C preprocessor > macros. This is a huge problem for people integrating diverse 3rd-party > libraries and Xerces-C, as the enum values often collide with macro > expressions. The result is thousands of lines of nearly indecipherable errors > being emitted from compilers. > Please change all enum values to "MixedCase". If you happen to be aware of > any other ALLCAPS identifiers in Xerces-C, change those too. > This is very critical. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
