diff options
author | Tor Lillqvist <tml@collabora.com> | 2018-06-07 10:31:46 +0300 |
---|---|---|
committer | Michael Meeks <michael.meeks@collabora.com> | 2018-06-07 15:08:29 +0200 |
commit | 9d4195b437b2a3a63a91380c36702fd2e2f43287 (patch) | |
tree | d7a982e3e2f3dc213c64ecfcce28ba8a0e003877 /extensions | |
parent | 9a809384f6210e15f90e131a15b88b105c912092 (diff) |
The XCollection loop index is one-based
Change-Id: I2de9e53b2a0d923d4136ee5e093baa955c8301da
Reviewed-on: https://gerrit.libreoffice.org/55409
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
Tested-by: Michael Meeks <michael.meeks@collabora.com>
Diffstat (limited to 'extensions')
-rw-r--r-- | extensions/source/ole/unoobjw.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/extensions/source/ole/unoobjw.cxx b/extensions/source/ole/unoobjw.cxx index 81c7a124df6b..7033f5c9eddc 100644 --- a/extensions/source/ole/unoobjw.cxx +++ b/extensions/source/ole/unoobjw.cxx @@ -2119,7 +2119,7 @@ public: ULONG nReturned = 0; while (nLeft > 0) { - if (mnIndex >= mxCollection->getCount()) + if (mnIndex > mxCollection->getCount()) { SAL_INFO("extensions.olebridge", this << "@CXEnumVariant::Next(" << celt << "): got " << nReturned << ": S_FALSE"); return S_FALSE; @@ -2154,7 +2154,7 @@ public: ULONG nSkipped = 0; while (celt > 0) { - if (mnIndex >= mxCollection->getCount()) + if (mnIndex > mxCollection->getCount()) { SAL_INFO("extensions.olebridge", this << "@CXEnumVariant::Skip(" << celt << "): skipped " << nSkipped << ": S_FALSE"); return S_FALSE; |