Merge pull request #173 from BrendanChou/patch-2

optimize can_cancel modifier
daifoundation · Nov 29, 2018 · 1660090 · 1660090
2 parents a9bf3af + 19fe671
commit 1660090
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/expiring_market.sol b/src/expiring_market.sol
@@ -43,7 +43,7 @@ contract ExpiringMarket is DSAuth, SimpleMarket {
     // after close, anyone can cancel an offer
     modifier can_cancel(uint id) {
         require(isActive(id));
-        require(isClosed() || (msg.sender == getOwner(id)));
+        require((msg.sender == getOwner(id)) || isClosed());
         _;
     }