[lmi-commits] [lmi] master a729e1a 4/7: Regularize argument order

[Greg Chicares]

branch: master
commit a729e1a50f792c5ae7595287ec2640b942ff24b1
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Regularize argument order
    
    Rewrote arguments of operator!=() and std::swap in the same order.
    Neither order is better in either case. Different orders would seem to
    suggest otherwise.
    
    Incidentally, there didn't seem to be any need for a blank line between
    the only two lines in a brace-enclosed block.
---
 mvc_controller.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/mvc_controller.cpp b/mvc_controller.cpp
index 830b528..03497cb 100644
--- a/mvc_controller.cpp
+++ b/mvc_controller.cpp
@@ -347,10 +347,9 @@ void MvcController::ConditionallyEnableItems
 
         wxWindowUpdateLocker u(&control);
 
-        if(items != cached_items)
+        if(cached_items != items)
             {
             std::swap(cached_items, items);
-
             itembox->Set(cached_items);
             }