Remove redundant extern modifier for function declarations

UniQP · vondele · commit 258c13ba8cc8 · 2023-01-01T12:26:36.000+01:00
Functions have external linkage by default, so there's no need to declare them extern. closes #4308 No functional change
diff --git a/src/position.h b/src/position.h
@@ -204,7 +204,7 @@ class Position {
   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
@@ -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
@@ -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 {
 
