#2074 closed Bugs (fixed)
Function named VERSION causes problem with configure tools
Reported by: | anonymous | Owned by: | Robert Ramey |
---|---|---|---|
Milestone: | Component: | serialization | |
Version: | Boost 1.35.0 | Severity: | Problem |
Keywords: | Cc: |
Description
Hi guys,
In file:
boost/archive/basic_xml_archive.hpp
There are many functions with names all in capital. I think that, already, is a bad idea. The problem is that one of them is named VERSION.
This is one of the -D flags generated by the automake/libtool suite and thus I cannot compile using the XML archive system without adding a fix (i.e. with -DVERSION="1.3" on the command line it just won't compile.) People at large won't have any control over that one.
At this time, what I do is add this at the beginning of the file:
#ifdef VERSION #undef VERSION #endif
That works for me, but that is not a solution.
I would think that the names need to be fixed. All capital names are reserved for macros and using those there is really bad. I would not be too surprised if some of these names would clash with #define's found in the MS-Windows header files too!
Thank you. Alexis Wilke
P.S. this is still a problem in the current SVN.
Change History (3)
comment:1 by , 14 years ago
Status: | new → assigned |
---|
comment:2 by , 14 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Fixed in trunk. Pending testing, will migrate to release for next version.