dbaccess/source/ui/browser/brwctrlr.cxx |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit 6250d512e5d3f05cb411d0816a18045422b86d50
Author:     Xisco Fauli <[email protected]>
AuthorDate: Fri Oct 18 16:25:27 2024 +0200
Commit:     Xisco Fauli <[email protected]>
CommitDate: Fri Oct 18 19:30:02 2024 +0200

    tdf#163486: PVS: variable is assigned values twice successively
    
    V519    The 'bParserSuccess' variable is assigned values twice 
successively. Perhaps this is a mistake. Check lines: 1941, 1944.
    
    Change-Id: I4dac8d01533706a5423fd0d75e5ccdc0d3cf00c4
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175161
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <[email protected]>

diff --git a/dbaccess/source/ui/browser/brwctrlr.cxx 
b/dbaccess/source/ui/browser/brwctrlr.cxx
index e415e6cb5719..3512b6462757 100644
--- a/dbaccess/source/ui/browser/brwctrlr.cxx
+++ b/dbaccess/source/ui/browser/brwctrlr.cxx
@@ -1938,7 +1938,6 @@ void SbaXDataBrowserController::Execute(sal_uInt16 nId, 
const Sequence< Property
             bool bParserSuccess = false;
             try
             {
-                bParserSuccess = false;
                 xParser->setOrder(OUString());
                 xParser->appendOrderByColumn(xField, bSortUp);
                 bParserSuccess = true;

Reply via email to