sc/source/ui/dbgui/scuiasciiopt.cxx | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
New commits:
commit c374d8ed86262c8b09c158d22ec7e4b9dee115dd
Author: Eike Rathke <er...@redhat.com>
AuthorDate: Thu Aug 19 19:00:38 2021 +0200
Commit: Eike Rathke <er...@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 <er...@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();
