From a924ca0db76079e3d95e885afa2eca4822eaa0d6 Mon Sep 17 00:00:00 2001 From: Tsuda Kageyu Date: Thu, 14 May 2015 11:20:35 +0900 Subject: [PATCH] Expand the internal buffer of ByteVector only if really needed. Add tests for all execution paths of ByteVector::resize(). --- taglib/toolkit/tbytevector.cpp | 9 ++++-- tests/test_bytevector.cpp | 51 ++++++++++++++++++++++++++++++---- 2 files changed, 52 insertions(+), 8 deletions(-) diff --git a/taglib/toolkit/tbytevector.cpp b/taglib/toolkit/tbytevector.cpp index bf6c525f..7aeaea92 100644 --- a/taglib/toolkit/tbytevector.cpp +++ b/taglib/toolkit/tbytevector.cpp @@ -702,8 +702,13 @@ ByteVector &ByteVector::resize(uint size, char padding) { if(size != d->length) { detach(); - d->data->data.resize(d->offset + d->length); - d->data->data.resize(d->offset + size, padding); + + if(size > d->data->data.size() - d->offset) + d->data->data.resize(d->offset + size); + + if(size > d->length) + ::memset(DATA(d) + d->offset + d->length, padding, size - d->length); + d->length = size; } diff --git a/tests/test_bytevector.cpp b/tests/test_bytevector.cpp index e9d43c05..de682084 100644 --- a/tests/test_bytevector.cpp +++ b/tests/test_bytevector.cpp @@ -42,6 +42,7 @@ class TestByteVector : public CppUnit::TestFixture CPPUNIT_TEST(testNumericCoversion); CPPUNIT_TEST(testReplace); CPPUNIT_TEST(testIterator); + CPPUNIT_TEST(testResize); CPPUNIT_TEST_SUITE_END(); public: @@ -123,12 +124,6 @@ public: CPPUNIT_ASSERT(i.containsAt(j, 5, 0)); CPPUNIT_ASSERT(i.containsAt(j, 6, 1)); CPPUNIT_ASSERT(i.containsAt(j, 6, 1, 3)); - - ByteVector k = ByteVector("0123456789").mid(3, 4); - k.resize(6, 'A'); - CPPUNIT_ASSERT_EQUAL(uint(6), k.size()); - CPPUNIT_ASSERT_EQUAL('6', k[3]); - CPPUNIT_ASSERT_EQUAL('A', k[4]); } void testFind1() @@ -340,6 +335,50 @@ public: CPPUNIT_ASSERT_EQUAL('3', *it4); } + void testResize() + { + ByteVector a = ByteVector("0123456789"); + ByteVector b = a.mid(3, 4); + b.resize(6, 'A'); + CPPUNIT_ASSERT_EQUAL(uint(6), b.size()); + CPPUNIT_ASSERT_EQUAL('6', b[3]); + CPPUNIT_ASSERT_EQUAL('A', b[4]); + CPPUNIT_ASSERT_EQUAL('A', b[5]); + b.resize(10, 'B'); + CPPUNIT_ASSERT_EQUAL(uint(10), b.size()); + CPPUNIT_ASSERT_EQUAL('6', b[3]); + CPPUNIT_ASSERT_EQUAL('B', b[6]); + CPPUNIT_ASSERT_EQUAL('B', b[9]); + b.resize(3, 'C'); + CPPUNIT_ASSERT_EQUAL(uint(3), b.size()); + CPPUNIT_ASSERT_EQUAL(-1, b.find('C')); + b.resize(3); + CPPUNIT_ASSERT_EQUAL(uint(3), b.size()); + + // Check if a and b were properly detached. + + CPPUNIT_ASSERT_EQUAL(uint(10), a.size()); + CPPUNIT_ASSERT_EQUAL('3', a[3]); + CPPUNIT_ASSERT_EQUAL('5', a[5]); + + // Special case that refCount == 1 and d->offset != 0. + + ByteVector c = ByteVector("0123456789").mid(3, 4); + c.resize(6, 'A'); + CPPUNIT_ASSERT_EQUAL(uint(6), c.size()); + CPPUNIT_ASSERT_EQUAL('6', c[3]); + CPPUNIT_ASSERT_EQUAL('A', c[4]); + CPPUNIT_ASSERT_EQUAL('A', c[5]); + c.resize(10, 'B'); + CPPUNIT_ASSERT_EQUAL(uint(10), c.size()); + CPPUNIT_ASSERT_EQUAL('6', c[3]); + CPPUNIT_ASSERT_EQUAL('B', c[6]); + CPPUNIT_ASSERT_EQUAL('B', c[9]); + c.resize(3, 'C'); + CPPUNIT_ASSERT_EQUAL(uint(3), c.size()); + CPPUNIT_ASSERT_EQUAL(-1, c.find('C')); + } + }; CPPUNIT_TEST_SUITE_REGISTRATION(TestByteVector); -- 2.40.0