make OUString::copy() clip (was: [PUSHED 3-6] resolved fdo#53012 crash in CSV fixed width import)

Eike Rathke <erack -AT- redhat.com>
Wed, 1 Aug 2012 19:05:35 +0200

Hi Stephan,

On Wednesday, 2012-08-01 18:17:12 +0200, Stephan Bergmann wrote:

make rtl::OUString::copy(beginIndex, count) clip to [0..length)?
yeah, why not  (then again, -1's sentinel nature, cf. indexOf, might
mean that silent clipping of beginIndex=-1 to beginIndex=0 is
unfortunate)


We may step into more of these traps in transitions from String to
OUString, so clipping IMHO is good.

I think beginIndex<0 or count<=0 should always return an empty string
and output a SAL_WARN, maybe also SAL_WARN if clipping occurred as that
may indicate a logic error.

  Eike

-- 
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3  9E96 2F1A D073 293C 05FD

Attachment: pgp7MNEKVVTx3.pgp
Description: PGP signature

Context

[REVIEW 3-6] resolved fdo#53012 crash in CSV fixed width import · Eike Rathke
- [PUSHED 3-6] resolved fdo#53012 crash in CSV fixed width import · Michael Meeks
  - Re: [PUSHED 3-6] resolved fdo#53012 crash in CSV fixed width import · Stephan Bergmann
    - make OUString::copy() clip (was: [PUSHED 3-6] resolved fdo#53012 crash in CSV fixed width import) · Eike Rathke
      - Re: make OUString::copy() clip · Michael Stahl
        
        Re: make OUString::copy() clip · Eike Rathke
        
        Re: make OUString::copy() clip · Stephan Bergmann
        
        Re: make OUString::copy() clip · Noel Grandin
        Re: make OUString::copy() clip · Stephan Bergmann
- Re: [PUSHED][REVIEW 3-6] resolved fdo#53012 crash in CSV fixed width import · Noel Power

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.