diff options
| author | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-07-31 16:00:02 +0200 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2018-08-02 11:07:55 +0200 |
| commit | 2f3684b2289a8c46dc6144064a452cc529400f28 (patch) | |
| tree | fc460328cc6963b9a156c23395ce8f15c70e878d /sal/qa/rtl | |
| parent | 4804a1474ccba8df57f0a0151bd69237e82eb618 (diff) | |
[API CHANGE] add some more asserts to the string functions
rtl_[u]String_alloc now requires that the length be >= 0.
Since this function is only @since Libreoffice 4.1, it is unlikely
to be widely used externally.
Removed some unit tests that were testing invalid or out of
range paramers, which are already not allowed according to the
documented contract of those functions.
The change in writerfilter is because the new asserts triggered
when running testFdo74745
The change in SwTextNode::EraseText is because testFdo60842
triggered the assert in replaceAt.
The change in SwFieldSlot::SwFieldSlot is because
testMoveRange::Import_Export_Import triggered the assert in
replaceAt.
The changes in SwFieldSlot::SwFieldSlot and TabControl::ImplGetItemSize
are due to failures in the uitests.
Change-Id: Ib317261067649b0de96df12873ce31360cd24681
Reviewed-on: https://gerrit.libreoffice.org/58390
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'sal/qa/rtl')
| -rw-r--r-- | sal/qa/rtl/oustring/rtl_OUString2.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sal/qa/rtl/oustring/rtl_OUString2.cxx b/sal/qa/rtl/oustring/rtl_OUString2.cxx index 426834b88f5f..7dce206300a2 100644 --- a/sal/qa/rtl/oustring/rtl_OUString2.cxx +++ b/sal/qa/rtl/oustring/rtl_OUString2.cxx @@ -584,7 +584,7 @@ public: // search for sun, start at the end, found (pos==0) rtl::OUString aStr("sun java system"); rtl::OUString aSearchStr("sun"); - lastIndexOf_oustring_offset(aStr, aSearchStr, -1, -1); + lastIndexOf_oustring_offset(aStr, aSearchStr, -1, 1); } void lastIndexOf_test_oustring_001() |