Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index


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


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.