Remove redundant extern modifier for function declarations

Functions have external linkage by default, so there's no need to
declare them extern.

closes https://github.com/official-stockfish/Stockfish/pull/4308

No functional change

diff --git a/src/position.h b/src/position.h
index 078ff5b79f23e9e2d85b943734f2c4713bbbb3bd..90247cb1823d97ce19ef9edba49999c7043d9810 100644 (file)
--- a/src/position.h
+++ b/src/position.h
@@ -204,7 +204,7 @@ private:
   bool chess960;
 };
 
-extern std::ostream& operator<<(std::ostream& os, const Position& pos);
+std::ostream& operator<<(std::ostream& os, const Position& pos);
 
 inline Color Position::side_to_move() const {
   return sideToMove;

diff --git a/src/psqt.h b/src/psqt.h
index 4ee0e379b145b5262ff53b413904813b229ecad2..bd78be90f232133d59d0c87069afd5f3036e8214 100644 (file)
--- a/src/psqt.h
+++ b/src/psqt.h
@@ -30,7 +30,7 @@ namespace Stockfish::PSQT
 extern Score psq[PIECE_NB][SQUARE_NB];
 
 // Fill psqt array from a set of internally linked parameters
-extern void init();
+void init();
 
 } // namespace Stockfish::PSQT
 

diff --git a/src/uci.cpp b/src/uci.cpp
index 5d842d25e73b6635b1472d2fead5eb525f602336..dc5f10e12f2d0ab434793791dcfa6ab73c89b5dc 100644 (file)
--- a/src/uci.cpp
+++ b/src/uci.cpp
@@ -36,7 +36,7 @@ using namespace std;
 
 namespace Stockfish {
 
-extern vector<string> setup_bench(const Position&, istream&);
+vector<string> setup_bench(const Position&, istream&);
 
 namespace {