diff options
| author | Tor Lillqvist <tml@collabora.com> | 2017-07-31 14:50:34 +0300 |
|---|---|---|
| committer | Tor Lillqvist <tml@collabora.com> | 2017-07-31 14:52:29 +0300 |
| commit | 2702c2d1eb9af2d03d5e0a1b7704ab170a5e8d31 (patch) | |
| tree | 3f1686c4e71d6eb33b9e6aeb3d05556ac4b7976b /sc | |
| parent | 76badcaf7a426458be55355427d6ef689d62ac69 (diff) | |
The code handles the nLevel == nCurLevel case just fine
I get the warning from such a case when loading an .xls written by
Excel 2013. I have no idea how common it is. I have no deeper
understanding of the code here. I just want to get rid of one more
unnecessary warning.
If nLevel > nCurLevel the code indeed probably will misbehave quite
badly (thanks to unsigned arithmetics).
Change-Id: I6cc5af454049e6ec054865535d2f7a54c6264613
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/filter/excel/exctools.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/filter/excel/exctools.cxx b/sc/source/filter/excel/exctools.cxx index 18a1f72ca2aa..85f759859329 100644 --- a/sc/source/filter/excel/exctools.cxx +++ b/sc/source/filter/excel/exctools.cxx @@ -131,7 +131,7 @@ void XclImpOutlineBuffer::MakeScOutline() } else { - OSL_ENSURE(nLevel < nCurLevel, "XclImpOutlineBuffer::MakeScOutline: unexpected level!"); + OSL_ENSURE(nLevel <= nCurLevel, "XclImpOutlineBuffer::MakeScOutline: unexpected level!"); for (sal_uInt8 i = 0; i < nCurLevel - nLevel; ++i) { if (aOutlineStack.empty()) |