[Libreoffice] [PUSHED] Re: [PATCH] Replace WW8Bytes with ww::bytes

Jan Holesovsky <kendy -AT- suse.cz>
Fri, 07 Oct 2011 00:37:17 +0200

Hi Maciej,

On 2011-10-05 at 22:02 +0200, Maciej Rumianowski wrote:

 That same would be allowed by 
class bytes : public std::vector<sal_uInt8>
{
public:
const sal_uInt8* data() const { return &front(); }
...

So if it is desired solution I can modify Patches?


No need to, I took this approach

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f1eda55a0670cd571d676b46d2ac19fe86d51a3f

and modified your patches myself.  I've pushed them under your name, as
all I did was a sed :-)

http://cgit.freedesktop.org/libreoffice/core/commit/?id=6cdc31837c73accca69ed9b1702e269f50c96bff 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=37836e3ba37c1ff7c303047a418dcbc3ef390afe
http://cgit.freedesktop.org/libreoffice/core/commit/?id=bcf7d383578a00b961a0484534d810202cf57852
http://cgit.freedesktop.org/libreoffice/core/commit/?id=d052ac384fbb476fe4525ab2d123d79b1158acb4

Thank you a lot,
Kendy

Context

[Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Maciej Rumianowski
- Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Jan Holesovsky
  - Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Lubos Lunak
    - Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Maciej Rumianowski
      - Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Michael Stahl
        
        Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Stephan Bergmann
        
        Re: [Libreoffice] [PATCH] Replace WW8Bytes with ww::bytes · Maciej Rumianowski
      - [Libreoffice] [PUSHED] Re: [PATCH] Replace WW8Bytes with ww::bytes · Jan Holesovsky
        
        Re: [Libreoffice] [PUSHED] Re: [PATCH] Replace WW8Bytes with ww::bytes · Maciej Rumianowski

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.