diff options
| author | Tor Lillqvist <tml@collabora.com> | 2018-06-07 15:53:06 +0300 |
|---|---|---|
| committer | Tor Lillqvist <tml@collabora.com> | 2018-06-08 17:55:36 +0200 |
| commit | 4328d181b670356c9dbd69f1ec1dedb473b59e8e (patch) | |
| tree | 7c05312372c6d7fbcd9b44b3150dc64e9f289ad8 /extensions | |
| parent | 8d43de9a88dc7c7fec0e7c794cef14953fb34b6e (diff) | |
Fix glitch in SAL_INFO logging
Change-Id: I421d3980c50706f927a0611e48c33a43267dd01b
Reviewed-on: https://gerrit.libreoffice.org/55477
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Tor Lillqvist <tml@collabora.com>
Diffstat (limited to 'extensions')
| -rw-r--r-- | extensions/source/ole/unoobjw.cxx | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/extensions/source/ole/unoobjw.cxx b/extensions/source/ole/unoobjw.cxx index 2ec57765b45e..cbe7992852df 100644 --- a/extensions/source/ole/unoobjw.cxx +++ b/extensions/source/ole/unoobjw.cxx @@ -2105,12 +2105,13 @@ public: return S_OK; } - virtual HRESULT STDMETHODCALLTYPE STDMETHODCALLTYPE Skip(ULONG celt) override + virtual HRESULT STDMETHODCALLTYPE STDMETHODCALLTYPE Skip(ULONG const celt) override { comphelper::Automation::AutomationInvokedZone aAutomationActive; + ULONG nLeft = celt; ULONG nSkipped = 0; - while (celt > 0) + while (nLeft > 0) { if (mnIndex > mxCollection->getCount()) { @@ -2118,7 +2119,7 @@ public: return S_FALSE; } mnIndex++; - celt--; + nLeft--; } SAL_INFO("extensions.olebridge", this << "@CXEnumVariant::Skip(" << celt << "): S_OK"); return S_OK; |