Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2

Regina Henschel <rb.henschel -AT- t-online.de>
Wed, 07 Sep 2011 15:04:47 +0200

Hi Eike,

I see a lot of sal_uInt64 in the code. Is that supported for Windows? Asfar as I know at least the MSVC Express has only 4Byte long.


Kind regards
Regina

Eike Rathke schrieb:

Hi Korrawit,

On Tuesday, 2011-09-06 21:52:46 +0700, Korrawit Pruegsanusak wrote:

And please s/interger/integer ;)


Did that in the commit.

Also, it seems that you haven't check the range of ishift yet, whether
it is between -48 and 48 (from your description), or it isn't
necessary? I don't have knowledge on this.


Actually those don't have to be restricted and I implemented another
algorithm to support larger values.

My last nitpick, what about the strings capitalization?


Also did that.

   Eike




_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Context

[Libreoffice] [Patch] new BITxxx functions for ODF 1.2 · libo
- Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Thorsten Behrens
  - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Olivier Hallot
    - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
      - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Olivier Hallot
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
  - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Niko Rönkkö
    - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Korrawit Pruegsanusak
      - Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Regina Henschel
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Stephan Bergmann
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Noel Grandin
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Tor Lillqvist
        
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Regina Henschel
        Re: [Libreoffice] Updated [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
- Re: [Libreoffice] [Patch] new BITxxx functions for ODF 1.2 · Eike Rathke
- Re: [Libreoffice] [PUSHED] new BITxxx functions for ODF 1.2 · Eike Rathke

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.