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 10:12:37 +0000 |
commit | 9f0096d934fcdec6cd55e32a15fd438f628f894b (patch) | |
tree | adf43114321dd956e70c50d5359c6e6787967854 /sd | |
parent | bd631f933cf2f5a515abc303c277c1197aa54200 (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>
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 29cd4fc83710..afd08350bc90 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 ); |