[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Greg Chicares
Subject: [lmi-commits] [lmi] master a729e1a 4/7: Regularize argument order
Date: Thu, 13 Jun 2019 20:39:01 -0400 (EDT)

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);

reply via email to

[Prev in Thread] Current Thread [Next in Thread]