diff options
author | Noel Grandin <noel@peralex.com> | 2016-10-03 10:07:41 +0200 |
---|---|---|
committer | Michael Meeks <michael.meeks@collabora.com> | 2016-10-03 15:09:34 +0000 |
commit | 85bd2c1145e23b67aafdfcc9c2ca8d720194fe97 (patch) | |
tree | 920589bdd85d41f249e5ef83dda50a069b1aec6d | |
parent | fd1a5710a98fba9966247b7d2e6120975262f0d6 (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>
-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 ec108a3afb0b..dd9f4947f395 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 ); |