[PATCH] OUStringBuffer::remove( int ) -> truncate( int )

Luboš Luňák (via_Code_Review) gerrit at gerrit.libreoffice.org
Thu Dec 6 09:56:36 PST 2012


Hi,

I have submitted a patch for review:

    https://gerrit.libreoffice.org/1256

To pull it, you can do:

    git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/56/1256/1

OUStringBuffer::remove( int ) -> truncate( int )

Although this may seem consistent with remove( int, int ), it is
in fact rather misleading API. The biggest offender is most probably
buffer.remove( 'a' ) , which definitely does not do what it suggests
to do.

Change-Id: I287619cd4b953228b93fa68fb381d66c344c3865
---
M sal/inc/rtl/ustrbuf.hxx
M svl/source/numbers/zformat.cxx
M xmloff/source/style/xmlnumfi.cxx
3 files changed, 10 insertions(+), 10 deletions(-)


--
To view, visit https://gerrit.libreoffice.org/1256
To unsubscribe, visit https://gerrit.libreoffice.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I287619cd4b953228b93fa68fb381d66c344c3865
Gerrit-PatchSet: 1
Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Owner: Luboš Luňák <l.lunak at suse.cz>
Gerrit-Reviewer: Norbert Thiebaud <nthiebaud at gmail.com>



More information about the LibreOffice mailing list