https://bugs.freedesktop.org/show_bug.cgi?id=65108
--- Comment #19 from Michael Meeks <michael.meeks@collabora.com> ---
The ESC resolution was:
Header includes <> vs "" etc. (Tml/Sberg)
+ easy-hack closure:
+ https://bugs.freedesktop.org/show_bug.cgi?id=65108
+ situation reasonably clear, MS compiler has a very odd use of ""
+ if we want to do a global cleanup, and be reasonably sane
we should only use <> (Sberg)
+ problems with own vs. system version (Norbert)
+ concern with local includes vs. public ones
+ local includes have no directory path anyway (Stephan)
+ in places we do relative includes "../foo/baa.h" (Kohei)
+ helps to cleanup pre-compiled headers (Sberg)
+ we should use <> everywhere without a relative path.
+ but it's fine to continue using "" for whatever corner
cases make sense,
identify and restore these post-facto (Kohei)
I guess scripting this is the way to go & do it in one big commit :-)
--
You are receiving this mail because:
You are on the CC list for the bug.
Context
- [Bug 65108] Clean-up header includes (global/local) · bugzilla-daemon
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.