[Libreoffice] [PATCH] Remove "using namespace ::rtl" in sc

[Bjoern Michaelsen]

On Thu, 24 Feb 2011 10:53:32 -0500
Kohei Yoshida <kyoshida at novell.com> wrote:

> On Thu, 2011-02-24 at 15:30 +0000, Caolán McNamara wrote:
> > 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.
> 
> Well, the task itself on the easy hack page *seems to imply* that
> removing rtl:: off of e.g. rtl::OUString is part of it.  BTW, I'm not
> the author of this task just to clarify. ;-)

Well, I was the original author of the task and I did not intend to
imply that one has to remove the rtl:: off of e.g. rtl::OUStrings. ;-)

Best Regards,

Bjoern

-- 
https://launchpad.net/~bjoern-michaelsen
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110224/051f1afd/attachment.pgp>