[Libreoffice] [PATCH] [PUSHED] Removed commented code

Kohei Yoshida kohei.yoshida at gmail.com
Fri Feb 18 16:40:09 PST 2011


Hi Christina,

On Fri, Feb 18, 2011 at 4:21 PM, Christina Roßmanith
<ChrRossmanith at web.de> wrote:
> Hi,
>
> I've removed some lines of commented code in
>
> libs-core/connectivity/source/drivers/file/
>  FResultSet.cxx
>  FStatement.cxx
>  fanalyzer.cxx
>  fcomp.cxx

Looks all good.  Pushed.

I had to change the format of one if-statement after the comment
removal, but other than that, the rest has been applied as-is.

> For fcomp.cxx I'm not sure if line 82 "// inline OCursor..." might be
> removed as well.

I agree.  Removed in my follow-up commit.

>
> There were some calls of RTL_LOGFILE_CONTEXT_AUTHOR commented but the
> majority wasn't. Can the commented calls be removed or should every method
> have it's call of RTL_LOGFILE_CONTEXT_AUTHOR?

Actually I've been wondering about these calls too.  They are
everywhere, yet it's not entirely clear what the purpose of these
calls are.  My personal preference is to remove them, but I would like
to at least know why they are there to begin with...

Kohei


More information about the LibreOffice mailing list