Retire Position::fast_copy()

It is never used and could be tricky, so remove it.

No functional change.

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

diff --git a/src/position.cpp b/src/position.cpp
index 7d01cb60f10a293c78d354b357b9cd3bc6a09be3..5b4dba70abc21fb7ad56c37dd0937ecfd24cbf28 100644 (file)
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -87,7 +87,7 @@ Position::Position() {}
 
 Position::Position(const Position& pos) {
 
-  fast_copy(pos);
+  memcpy(this, &pos, sizeof(Position));
   detach(); // Always detach() in copy c'tor to avoid surprises
 }
 
@@ -97,19 +97,6 @@ Position::Position(const string& fen) {
 }
 
 
-/// Position::fast_copy() creates a partial copy of the given position,
-/// only data that changes with a do_move() / undo_move() cycle is copied,
-/// in particular for stateInfo are copied only the pointers, so that the
-/// actual data remains stored in the parent Position. This is not a problem
-/// if the parent Position is known not to be destroyed while we are still alive,
-/// as is the common case, see detach() otherwise.
-
-void Position::fast_copy(const Position& pos) {
-
-  memcpy(this, &pos, sizeof(Position));
-}
-
-
 /// Position::detach() copies the content of the current state and castling
 /// masks inside the position itself. This is needed when the st pointee could
 /// become stale, as example because the caller is about to going out of scope.

diff --git a/src/position.h b/src/position.h
index c946cb6ab789030dd7bed423000b7b864e6085d8..0fce1860991eaec247937736062de163d2091b53 100644 (file)
--- a/src/position.h
+++ b/src/position.h
@@ -156,7 +156,6 @@ public:
   void print(Move m = MOVE_NONE) const;
 
   // Copying
-  void fast_copy(const Position& pos);
   void flipped_copy(const Position& pos);
 
   // The piece on a given square