Avoid friend

operator<<(os, pos) does not need to access any private members of pos.

No functional change.

Resolves #492

diff --git a/src/position.cpp b/src/position.cpp
index c432bb90e9f51fcf93d7f246a04b98916aeb4366..0be3301a9e09467e9bc4970255f2e52c05696dcb 100644 (file)
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -118,7 +118,7 @@ std::ostream& operator<<(std::ostream& os, const Position& pos) {
   }
 
   os << "\nFen: " << pos.fen() << "\nKey: " << std::hex << std::uppercase
-     << std::setfill('0') << std::setw(16) << pos.st->key << std::dec << "\nCheckers: ";
+     << std::setfill('0') << std::setw(16) << pos.key() << std::dec << "\nCheckers: ";
 
   for (Bitboard b = pos.checkers(); b; )
       os << UCI::square(pop_lsb(&b)) << " ";

diff --git a/src/position.h b/src/position.h
index 0d34aba44fe1c9e8fe74ba71dc4532e03f975cbc..0f1620b115600e0402705075e737bb43fbd96b6e 100644 (file)
--- a/src/position.h
+++ b/src/position.h
@@ -82,8 +82,6 @@ struct StateInfo {
 
 class Position {
 
-  friend std::ostream& operator<<(std::ostream&, const Position&);
-
 public:
   static void init();
 
@@ -210,6 +208,8 @@ private:
   bool chess960;
 };
 
+extern std::ostream& operator<<(std::ostream& os, const Position& pos);
+
 inline Color Position::side_to_move() const {
   return sideToMove;
 }