Re: [Libreoffice] [PUSHED] Re: [PATCH 11/12] RTL_CONSTASCII_USTRINGPARAM components - zipapi

2010-11-18 Thread Caolán McNamara
On Wed, 2010-11-17 at 21:58 +0100, Pierre-André Jacquod wrote: Hello, I saw it too, this allow me to rize a question. My firt though was to remove this one - since commentted out - but I left it there not knowing if this was a way to handle this here.? For my taste, I would have remove it.

[Libreoffice] [PUSHED] Re: [PATCH 11/12] RTL_CONSTASCII_USTRINGPARAM components - zipapi

2010-11-17 Thread Caolán McNamara
On Tue, 2010-11-16 at 22:50 +0100, Pierre-André Jacquod wrote: regards heh, I had a bit of a moment when I saw file:///d:/777/Encrypted/picture in there before I realized this is someones debugging code with an if (0) around it. Anyway, that's beside the point. Your changes were good, thanks

Re: [Libreoffice] [PUSHED] Re: [PATCH 11/12] RTL_CONSTASCII_USTRINGPARAM components - zipapi

2010-11-17 Thread Pierre-André Jacquod
Hello, I saw it too, this allow me to rize a question. My firt though was to remove this one - since commentted out - but I left it there not knowing if this was a way to handle this here.? For my taste, I would have remove it. - Debug it at home, not on public branch... Your input? regards On