[Libreoffice] [PATCH] Removed dependencies on tools/solar.h
Kohei Yoshida
kyoshida at novell.com
Mon Jan 3 20:36:08 PST 2011
On Mon, 2011-01-03 at 21:47 +0100, Soeren Moeller wrote:
> Hi
>
> I have removed dependencies on tools/solar.h in some files in sc
> (according to http://wiki.documentfoundation.org/Easy_Hacks#write_tools.2F_pieces_out
> ) please review and commit.
Thanks, pushed!
BTW, we generally prefer the standard bool over sal_Bool, so I replaced
sal_Bool with bool in your patch. The only place we need to use
sal_Bool is when dealing with the UNO API. Other than that, the
standard boolean type is preferred.
Also, it's a bit weird to use sal_uIntPtr which isn't used much in our
code base. So I replaced that with sal_uInt32.
Kohei
--
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>
More information about the LibreOffice
mailing list