diff options
author | Noel Grandin <noel@peralex.com> | 2016-10-03 10:07:41 +0200 |
---|---|---|
committer | Andras Timar <andras.timar@collabora.com> | 2016-10-04 11:24:47 +0000 |
commit | b41b72139ee5a92260b5d8732a437106d31b1688 (patch) | |
tree | dd37647070ccf0faf8fa3498cd23dfb400b3f8b2 /sd | |
parent | 1922c8b1d46e0067ed21a18d249f04da6f3b0160 (diff) |
fix bug in DocumentSettings::LoadList when testing result of lastIndexOf
managed to trigger it in a flat ODF test document of mine.
I suspect that this might have something to do with one of the various
conversions to OUString, the old String APIs might have returned -LARGE_INTEGER
instead of -1.
code was introduced in commit
cbcfda9b2079ea4ef83b2a42828408b5f70f7692
Author: Michael Meeks <michael.meeks@novell.com>
Date: Mon Sep 5 17:22:24 2011 +0100
add XPropertyList enum, factory, and associated cleanup
Change-Id: I39b4715cc12ef6366fe0466786589ef198602a98
Reviewed-on: https://gerrit.libreoffice.org/29473
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
(cherry picked from commit 9f0096d934fcdec6cd55e32a15fd438f628f894b)
Reviewed-on: https://gerrit.libreoffice.org/29480
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 85bd2c1145e23b67aafdfcc9c2ca8d720194fe97)
Reviewed-on: https://gerrit.libreoffice.org/29508
Reviewed-by: Andras Timar <andras.timar@collabora.com>
Tested-by: Andras Timar <andras.timar@collabora.com>
Diffstat (limited to 'sd')
-rw-r--r-- | sd/source/ui/unoidl/UnoDocumentSettings.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sd/source/ui/unoidl/UnoDocumentSettings.cxx b/sd/source/ui/unoidl/UnoDocumentSettings.cxx index d3b506bde4fb..c2166f62f304 100644 --- a/sd/source/ui/unoidl/UnoDocumentSettings.cxx +++ b/sd/source/ui/unoidl/UnoDocumentSettings.cxx @@ -233,7 +233,7 @@ bool DocumentSettings::LoadList( XPropertyListType t, const OUString &rInPath, sal_Int32 nSlash = rInPath.lastIndexOf('/'); OUString aPath, aName; - if (nSlash < -1) + if (nSlash < 0) aName = rInPath; else { aName = rInPath.copy( nSlash + 1 ); |