Merge lp:~verifydtapn-contributers/verifydtapn/SlowHeuristicsBug into lp:verifydtapn

Proposed by Peter Gjøl Jensen
Status: Merged
Approved by: Jiri Srba
Approved revision: 309
Merged at revision: 309
Proposed branch: lp:~verifydtapn-contributers/verifydtapn/SlowHeuristicsBug
Merge into: lp:verifydtapn
Diff against target: 38 lines (+4/-4)
1 file modified
src/DiscreteVerification/SearchStrategies/WeightQueryVisitor.cpp (+4/-4)
To merge this branch: bzr merge lp:~verifydtapn-contributers/verifydtapn/SlowHeuristicsBug
Reviewer Review Type Date Requested Status
Jiri Srba Approve
Review via email: mp+225713@code.launchpad.net

Description of the change

Fixed the linked bug. The fix is trivial.

To post a comment you must log in.
Revision history for this message
Jiri Srba (srba) wrote :

Tested and works. The fix is clear, so no need for more reviews here.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/DiscreteVerification/SearchStrategies/WeightQueryVisitor.cpp'
2--- src/DiscreteVerification/SearchStrategies/WeightQueryVisitor.cpp 2014-02-16 19:20:25 +0000
3+++ src/DiscreteVerification/SearchStrategies/WeightQueryVisitor.cpp 2014-07-04 20:51:42 +0000
4@@ -55,7 +55,7 @@
5 IntResult left;
6 expr.getLeft().accept(*this, left);
7 IntResult right;
8- expr.getLeft().accept(*this, right);
9+ expr.getRight().accept(*this, right);
10 static_cast<IntResult&>(context).value
11 = compare(left.value, expr.getOperator(), right.value);
12 }
13@@ -72,7 +72,7 @@
14 IntResult left;
15 expr.getLeft().accept(*this, left);
16 IntResult right;
17- expr.getLeft().accept(*this, right);
18+ expr.getRight().accept(*this, right);
19 ((IntResult&)context).value = left.value * right.value;
20 }
21
22@@ -86,7 +86,7 @@
23 IntResult left;
24 expr.getLeft().accept(*this, left);
25 IntResult right;
26- expr.getLeft().accept(*this, right);
27+ expr.getRight().accept(*this, right);
28 ((IntResult&)context).value = left.value - right.value;
29 }
30
31@@ -94,7 +94,7 @@
32 IntResult left;
33 expr.getLeft().accept(*this, left);
34 IntResult right;
35- expr.getLeft().accept(*this, right);
36+ expr.getRight().accept(*this, right);
37 ((IntResult&)context).value = left.value + right.value;
38 }
39

Subscribers

People subscribed via source and target branches