[Libreoffice] [PATCH] Remove "using namespace ::rtl" in sc
Caolán McNamara
caolanm at redhat.com
Thu Feb 24 07:30:45 PST 2011
On Thu, 2011-02-24 at 10:11 -0500, Kohei Yoshida wrote:
> On Thu, 2011-02-24 at 15:26 +0100, Julien Nabet wrote:
> > 2) must we remove all the "rtl::" stuff ?
>
> If you mean "using namespace rtl;", yes, because that could cause
> conflict between com::sun::star::uno::Reference and rtl::Reference. I
> prefer we don't have any of those lines in our code base.
Just to clarify here, and looking at the patch. The goal is to remove
"using namespace rtl;", right ?
not to remove rtl:: off the front of everything.
i.e. in the attached patch I see
-using namespace rtl;
which looks fine,
and I see various
+using ::rtl::OUString
which looks fine when necessary to make it possible to compile a file
which used to have "using namespace rtl;" and uses bare "OUString"
without making vast amounts of changes.
but I see some
-rtl::OUString
+OUString
which would be unnecessary, and likely cause a huge amount of churn,
no ?
C.
More information about the LibreOffice
mailing list