Prefer empty() to size()

As Heinz says:

"Function empty() should have a constant run-time even
 on lousy compilers and you spare the not.

The change is even measurable: + 100-150 nodes/sec. Wow:-)"

Patch from Heinz van Saanen

No functional change.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>

diff --git a/src/search.cpp b/src/search.cpp
index 77acd89e39b12310d5333708de19c5c85c7a883f..3cae0e3fa2095c10ab08378cb142a3be37cc68cf 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -507,7 +507,7 @@ namespace {
     Rml.init(pos, rootMoves);
 
     // Handle special case of searching on a mate/stalemate position
     Rml.init(pos, rootMoves);
 
     // Handle special case of searching on a mate/stalemate position

-    if (!Rml.size())
+    if (Rml.empty())

     {
         cout << "info" << depth_to_uci(DEPTH_ZERO)
              << score_to_uci(pos.in_check() ? -VALUE_MATE : VALUE_DRAW, alpha, beta) << endl;
     {
         cout << "info" << depth_to_uci(DEPTH_ZERO)
              << score_to_uci(pos.in_check() ? -VALUE_MATE : VALUE_DRAW, alpha, beta) << endl;