Update checkother.cpp

chrchr-github · web-flow · commit b4ebcb47d420 · 2026-07-14T14:20:50.000+02:00
diff --git a/lib/checkother.cpp b/lib/checkother.cpp
@@ -3056,9 +3056,11 @@ void CheckOtherImpl::checkDuplicateExpression()
                         }
                         if (tok->str() != "=") {
                             const Token* par = tok->astParent();
-                            while (par && tok->str() == par->str() && precedes(par, tok)) { // chain of identical operators with parentheses
+                            while (par && tok->str() == par->str() && precedes(par->astOperand1(), tok)) { // chain of identical operators with parentheses
                                 checkDuplicate(par->astOperand1(), tok->astOperand1(), par);
                                 checkDuplicate(par->astOperand1(), tok->astOperand2(), par);
+                                checkDuplicate(par->astOperand2(), tok->astOperand1(), par);
+                                checkDuplicate(par->astOperand2(), tok->astOperand2(), par);
                                 par = par->astParent();
                             }
                         }

Original file line number	Diff line number	Diff line change
`@@ -3056,9 +3056,11 @@ void CheckOtherImpl::checkDuplicateExpression()`
`3056`	`3056`	`}`
`3057`	`3057`	`if (tok->str() != "=") {`
`3058`	`3058`	`const Token* par = tok->astParent();`
`3059`		`- while (par && tok->str() == par->str() && precedes(par, tok)) { // chain of identical operators with parentheses`
	`3059`	`+ while (par && tok->str() == par->str() && precedes(par->astOperand1(), tok)) { // chain of identical operators with parentheses`
`3060`	`3060`	`checkDuplicate(par->astOperand1(), tok->astOperand1(), par);`
`3061`	`3061`	`checkDuplicate(par->astOperand1(), tok->astOperand2(), par);`
	`3062`	`+ checkDuplicate(par->astOperand2(), tok->astOperand1(), par);`
	`3063`	`+ checkDuplicate(par->astOperand2(), tok->astOperand2(), par);`
`3062`	`3064`	`par = par->astParent();`
`3063`	`3065`	`}`
`3064`	`3066`	`}`