commit c374d8ed86262c8b09c158d22ec7e4b9dee115dd
author Eike Rathke <erack@redhat.com> Thu Aug 19 19:00:38 2021 +0200
committer Eike Rathke <erack@redhat.com> Thu Aug 19 19:51:08 2021 +0200
tree b4dd859af3b50856df4853a8cc11e454ab18f0e3
parent 9c13f1ad6384ee05a10ed688b93a46f6489a872a

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@redhat.com>
Tested-by: Jenkins

sc/source/ui/dbgui/scuiasciiopt.cxx

diff --git a/sc/source/ui/dbgui/scuiasciiopt.cxx b/sc/source/ui/dbgui/scuiasciiopt.cxx
index a95d8bd..3ed0b05 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();