diff options
| author | Eike Rathke <erack@redhat.com> | 2021-10-06 21:25:18 +0200 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2021-10-07 12:52:49 +0200 |
| commit | 14f81ca6ad289bf86781f642aa129b33151247cf (patch) | |
| tree | bd950a3377c0d968464c35a279af70ab9ae375f1 /sc | |
| parent | 1ef671c5a2f4b629ba6e280c5036ba781cc8787c (diff) | |
Related: tdf#144970 == ScRefFlags::ZERO is not a replacement for != 0 ...
It still worked by chance because if (!bValid) the string is
parsed with ScRangeList, which does deliver a result.
That will have to be adapted to the then expected UI
representation though in which case it wouldn't work.
Fallout from
commit 51d0b4037b36a4a74c9181b95edb18017542ab79
CommitDate: Sat Mar 12 00:21:29 2016 +0000
tdf#84938 Change defines to typed_flags
Change-Id: I9612b34829c12e59d4b390c6aa035aca84e87c73
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123194
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/ui/docshell/docsh4.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sc/source/ui/docshell/docsh4.cxx b/sc/source/ui/docshell/docsh4.cxx index 6ccf37a924b4..951f0fe96823 100644 --- a/sc/source/ui/docshell/docsh4.cxx +++ b/sc/source/ui/docshell/docsh4.cxx @@ -380,7 +380,7 @@ void ScDocShell::Execute( SfxRequest& rReq ) } ScAddress::Details aDetails(rDoc.GetAddressConvention(), 0, 0); - bool bValid = (aSingleRange.ParseAny(aRangeName, rDoc, aDetails) & ScRefFlags::VALID) == ScRefFlags::ZERO; + bool bValid = (aSingleRange.ParseAny(aRangeName, rDoc, aDetails) & ScRefFlags::VALID) != ScRefFlags::ZERO; if (!bValid) { aRangeListRef = new ScRangeList; |