Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-07 Thread Norbert Thiebaud
On Wed, Jan 5, 2011 at 12:56 PM, Soeren Moeller wrote: > Hi and thank you for your comments > > 2011/1/5 Kohei Yoshida : >> Hi Sören, >>> >>> +#include >>>  #include "address.hxx" >> >> This should not be necessary (though it won't hurt) as the address.hxx >> already includes sal types.  Does it

Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-06 Thread Kohei Yoshida
On Thu, 2011-01-06 at 11:27 +, Michael Meeks wrote: > On Wed, 2011-01-05 at 19:56 +0100, Soeren Moeller wrote: > > It is not an attachment as I used "git send-email" > > Heh - the kernel guys prefer in-line patches it seems; they find them > easier to review and comment on that way; per

Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-06 Thread Michael Meeks
On Wed, 2011-01-05 at 19:56 +0100, Soeren Moeller wrote: > It is not an attachment as I used "git send-email" Heh - the kernel guys prefer in-line patches it seems; they find them easier to review and comment on that way; personally - I don't care, its easy enough to save a file as an mb

Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-05 Thread Soeren Moeller
Hi and thank you for your comments 2011/1/5 Kohei Yoshida : > Hi Sören, > > Thanks for the patch.  Is there a reason why this one is not an > attachment? :-) It is not an attachment as I used "git send-email" to send it to the mailing list, but the result seems a bit wierd when reading the list, a

Re: [Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-05 Thread Kohei Yoshida
Hi Sören, Thanks for the patch. Is there a reason why this one is not an attachment? :-) Anyway, I have some comments. On Tue, 2011-01-04 at 20:20 +0100, Sören Möller wrote: > Replaced datatypes from tools/solar.h with corresponding types from > sal/types.h in drawpage.hxx/.cxx and added missi

[Libreoffice] [PATCH] Removed dependencies on tools/solar.h in drawpage.hxx/.cxx

2011-01-04 Thread Sören Möller
Replaced datatypes from tools/solar.h with corresponding types from sal/types.h in drawpage.hxx/.cxx and added missing include of sal/types.h in docparam.hxx --- sc/inc/docparam.hxx |1 + sc/inc/drawpage.hxx |4 ++-- sc/source/core/data/drawpage.cxx |2 +- 3