made it more readable

3 files changed, 11 insertions, 6 deletions

diff --git a/include/chess_board.hpp b/include/chess_board.hpp
index c277618..ac96323 100644
--- a/include/chess_board.hpp
+++ b/include/chess_board.hpp
@@ -28,11 +28,11 @@ class ChessBoard {
 public:
 	ChessBoard();
 
-	void populateBoard();
-	void printBoard();
+	bool populateBoard();
+	bool printBoard();
 protected:
 private:
-	static const unsigned BOARD_SIZE = 64;
+	static const std::size_t BOARD_SIZE = 64;
 
 	boardVector board;
 };
diff --git a/include/chess_piece.hpp b/include/chess_piece.hpp
index e7f2f79..5b2c1c2 100644
--- a/include/chess_piece.hpp
+++ b/include/chess_piece.hpp
@@ -30,6 +30,8 @@ class Knight;
 class Pawn;
 class EmptyPiece;
 
+typedef unsigned int boardPosition;
+
 // Colour enum class so that we can have the colours in the Chess Piece and not
 // using normal enums because of undefined errors
 
@@ -54,7 +56,7 @@ public:
 protected:
 	// current location of the piece which is protected as it can still be
 	// inherited by the piece classes
-	int m_x, m_y;
+	boardPosition m_x, m_y;
 
 	// defines what colour the piece is
 	Colour m_colour;
diff --git a/src/chess_board.cpp b/src/chess_board.cpp
index 3432b52..051decd 100644
--- a/src/chess_board.cpp
+++ b/src/chess_board.cpp
@@ -22,12 +22,13 @@ ChessBoard::ChessBoard() {
 		board.emplace_back(new EmptyPiece);
 }
 
-void ChessBoard::populateBoard() {
+bool ChessBoard::populateBoard() {
 	for(auto&& piece : board)
 		piece = chessPiecePtr(new Knight);
+	return true;
 }
 
-void ChessBoard::printBoard() {
+bool ChessBoard::printBoard() {
 	int i = 0;
 
 	for(auto&& piece : board) {
@@ -35,4 +36,6 @@ void ChessBoard::printBoard() {
 		piece->move(1, 2);
 		++i;
 	}
+
+	return true;
 }