On Thu, Feb 14, 2013 at 02:43:16PM +0000, Caolán McNamara <caolanm@redhat.com> wrote:
My best theory is that it was intended to speed up builds by having less "stuff" in the headers for compilers to parse by having the preprocessor strip out the stuff includers didn't need to see. Which would suggest that it will only be found around private methods, i.e. not protected/public methods or anything that takes up space.
I think they tried to avoid including headers for enumerations this way -- given that in C++ it's not possible to declare enumerations as an incomplete type, AFAIK.
So, IMO it's not necessary and feel free to remove it.
Agreed, better to remove these, those -- usually small -- headers containing the enumerations should not really make the build longer.
Attachment:
signature.asc
Description: Digital signature