[spirit1] including type_traits.hpp

When looking at <http://thread.gmane.org/gmane.comp.lib.boost.devel/180769>, i noticed that <boost\spirit\home\classic\core\parser.hpp> includes <boost/type_traits.hpp>, when only part of it (remove_reference.hpp?) actually seems to be required. Not really a problem, but including the full type_traits.hpp does add quite a bit of weight to my precompiled header.

When looking at <http://thread.gmane.org/gmane.comp.lib.boost.devel/180769>, i noticed that <boost\spirit\home\classic\core\parser.hpp> includes <boost/type_traits.hpp>, when only part of it (remove_reference.hpp?) actually seems to be required.
Not really a problem, but including the full type_traits.hpp does add quite a bit of weight to my precompiled header.
Do you care to submit a full patch? I suspect that simply changing the included file here will break other places relying on the fact that all type traits are included already (but the tests will tell...). Thanks! Regards Hartmut

Hartmut Kaiser <hartmut.kaiser <at> gmail.com> writes:
I suspect that simply changing the included file here will break other places relying on the fact that all type traits are included already (but the tests will tell...).
Simply changing the #include <boost/type_traits.hpp> to #include <boost/type_traits/remove_reference.hpp> Doesn't seem to cause any Spirit test failures on VC8/9 (i don't have any other compilers to hand). SideNote: I saw the error mentioned @ <http://thread.gmane.org/gmane.comp.parsers.spirit.general/13449> when running the regression tests. Any ideas about that?
participants (2)
-
Hartmut Kaiser
-
Richard Webb