Merge pull request #7457 from staabm/patch-2

Remove unnecessary abs() calls
main
Rob 6 years ago committed by GitHub
commit b350c09c7e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -100,7 +100,7 @@ class Solver
$literals = $rule->getLiterals(); $literals = $rule->getLiterals();
$literal = $literals[0]; $literal = $literals[0];
if (!$this->decisions->decided(abs($literal))) { if (!$this->decisions->decided($literal)) {
$this->decisions->decide($literal, 1, $rule); $this->decisions->decide($literal, 1, $rule);
continue; continue;
} }
@ -792,14 +792,14 @@ class Solver
// //
foreach ($literals as $literal) { foreach ($literals as $literal) {
if ($literal <= 0) { if ($literal <= 0) {
if (!$this->decisions->decidedInstall(abs($literal))) { if (!$this->decisions->decidedInstall($literal)) {
continue 2; // next rule continue 2; // next rule
} }
} else { } else {
if ($this->decisions->decidedInstall(abs($literal))) { if ($this->decisions->decidedInstall($literal)) {
continue 2; // next rule continue 2; // next rule
} }
if ($this->decisions->undecided(abs($literal))) { if ($this->decisions->undecided($literal)) {
$decisionQueue[] = $literal; $decisionQueue[] = $literal;
} }
} }

Loading…
Cancel
Save