[Libreoffice-commits] core.git: sc/source

[Eike Rathke (via logerrit)]

 sc/source/ui/dbgui/scuiasciiopt.cxx |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit c374d8ed86262c8b09c158d22ec7e4b9dee115dd
Author:     Eike Rathke <erack at redhat.com>
AuthorDate: Thu Aug 19 19:00:38 2021 +0200
Commit:     Eike Rathke <erack at redhat.com>
CommitDate: Thu Aug 19 19:51:08 2021 +0200

    Make the logic more logical.., tdf#102846 follow-up
    
    Same functionality better readable.
    
    Change-Id: Idb891a6e2f0a35528d6a1862e5743a22772eec11
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/120754
    Reviewed-by: Eike Rathke <erack at redhat.com>
    Tested-by: Jenkins

diff --git a/sc/source/ui/dbgui/scuiasciiopt.cxx b/sc/source/ui/dbgui/scuiasciiopt.cxx
index a95d8bdfff7a..3ed0b05cf677 100644
--- a/sc/source/ui/dbgui/scuiasciiopt.cxx
+++ b/sc/source/ui/dbgui/scuiasciiopt.cxx
@@ -801,11 +801,10 @@ IMPL_LINK_NOARG(ScImportAsciiDlg, UpdateTextHdl, ScCsvTableBox&, void)
     // when the dialog wasn't already presented to the user.
     // As a side effect this has the benefit that the check is only done on the
     // first set of visible lines.
-    sal_Unicode cDetectSep = ((mbDetectSep && !mxRbFixed->get_active()
+    mbDetectSep = (mbDetectSep && !mxRbFixed->get_active()
                 && (!mxCkbTab->get_active() || !mxCkbSemicolon->get_active()
-                    || !mxCkbComma->get_active() || !mxCkbSpace->get_active())) ? 0 : 0xffff);
-    if (cDetectSep == 0xffff)
-        mbDetectSep = false;
+                    || !mxCkbComma->get_active() || !mxCkbSpace->get_active()));
+    sal_Unicode cDetectSep = (mbDetectSep ? 0 : 0xffff);
 
     sal_Int32 nBaseLine = mxTableBox->GetGrid().GetFirstVisLine();
     sal_Int32 nRead = mxTableBox->GetGrid().GetVisLineCount();