From b11fb967a5f7da23d96575d114f7cc095758ea39 Mon Sep 17 00:00:00 2001 From: KazApps Date: Mon, 29 Jul 2024 21:45:24 +0900 Subject: [PATCH 1/3] =?UTF-8?q?types.h=E3=81=A8types.cpp=E3=81=A7=E5=86=97?= =?UTF-8?q?=E9=95=B7=E3=81=AA=E3=82=B3=E3=83=BC=E3=83=89=E3=82=92=E6=95=B4?= =?UTF-8?q?=E7=90=86=E3=81=97=E3=81=9F=E3=80=82?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- source/types.cpp | 2 +- source/types.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/source/types.cpp b/source/types.cpp index f7952fd6b..67fdd0d85 100644 --- a/source/types.cpp +++ b/source/types.cpp @@ -139,7 +139,7 @@ namespace Search { pos.undo_move(pv[0]); return false; - FOUND:; + FOUND: pos.undo_move(pv[0]); pv.push_back(m); // std::cout << m << std::endl; diff --git a/source/types.h b/source/types.h index 529f7c9f9..68d513e09 100644 --- a/source/types.h +++ b/source/types.h @@ -365,7 +365,7 @@ namespace Effect8 static bool aligned(Square sq1, Square sq2, Square sq3/* is ksq */) { auto d1 = Effect8::directions_of(sq1, sq3); - return d1 ? d1 == Effect8::directions_of(sq2, sq3) : false; + return d1 && d1 == Effect8::directions_of(sq2, sq3); } // -------------------- @@ -535,7 +535,7 @@ enum Piece : uint32_t // USIプロトコルで駒を表す文字列を返す。 // 駒打ちの駒なら先頭に"D"。 -static std::string usi_piece(Piece pc) { return std::string((pc & 32) ? "D":"") +static std::string usi_piece(Piece pc) { return (pc & 32) ? "D":"" + std::string(USI_PIECE).substr((pc & 31) * 2, 2); } // 駒に対して、それが先後、どちらの手番の駒であるかを返す。 From 05a52277eda0ccb6c707e4f2ad5c671a2c81ea23 Mon Sep 17 00:00:00 2001 From: KazApps Date: Tue, 6 Aug 2024 12:34:58 +0900 Subject: [PATCH 2/3] =?UTF-8?q?=E7=A9=BA=E7=99=BD=E3=82=84=E3=82=A4?= =?UTF-8?q?=E3=83=B3=E3=83=87=E3=83=B3=E3=83=88=E3=82=92=E5=BE=AE=E8=AA=BF?= =?UTF-8?q?=E6=95=B4?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- source/bitboard.cpp | 2 +- source/types.h | 22 +++++++++++----------- 2 files changed, 12 insertions(+), 12 deletions(-) diff --git a/source/bitboard.cpp b/source/bitboard.cpp index 8dee9d110..7f599f909 100644 --- a/source/bitboard.cpp +++ b/source/bitboard.cpp @@ -151,7 +151,7 @@ void Bitboards::init() // dirの方角に壁にぶつかる(盤外)まで延長していく。このとき、sq1から見てsq2のDirectionsは (1 << dir)である。 auto delta = Effect8::DirectToDeltaWW(dir); for (auto sq2 = to_sqww(sq1) + delta; is_ok(sq2); sq2 += delta) - Effect8::direc_table[sq1][sqww_to_sq(sq2)] = Effect8::to_directions(dir); + Effect8::direc_table[sq1][sqww_to_sq(sq2)] = Effect8::to_directions(dir); } diff --git a/source/types.h b/source/types.h index 68d513e09..676f0d19d 100644 --- a/source/types.h +++ b/source/types.h @@ -129,7 +129,7 @@ static std::ostream& operator<<(std::ostream& os, Rank r) { os << (char)('a' + r // 盤上の升目に対応する定数。 // 盤上右上(１一が0)、左下(９九)が80 // 方角を表現するときにマイナスの値を使うので符号型である必要がある。 -enum Square: int32_t +enum Square : int32_t { // 以下、盤面の右上から左下までの定数。 // これを定義していなくとも問題ないのだが、デバッガでSquare型を見たときに @@ -267,7 +267,7 @@ static std::ostream& operator<<(std::ostream& os, Square sq) { os << file_of(sq) // bit 14..18 : いまの升から盤外まで何升上に(略 // bit 19..23 : いまの升から盤外まで何升下に(略 // bit 24..28 : いまの升から盤外まで何升左に(略 -enum SquareWithWall: int32_t { +enum SquareWithWall : int32_t { // 相対移動するときの差分値 SQWW_R = SQ_R - (1 << 9) + (1 << 24), SQWW_U = SQ_U - (1 << 14) + (1 << 19), SQWW_D = -int(SQWW_U), SQWW_L = -int(SQWW_R), SQWW_RU = int(SQWW_R) + int(SQWW_U), SQWW_RD = int(SQWW_R) + int(SQWW_D), SQWW_LU = int(SQWW_L) + int(SQWW_U), SQWW_LD = int(SQWW_L) + int(SQWW_D), @@ -306,7 +306,7 @@ namespace Effect8 // 方角を表す。遠方駒の利きや、玉から見た方角を表すのに用いる。 // bit0..右上、bit1..右、bit2..右下、bit3..上、bit4..下、bit5..左上、bit6..左、bit7..左下 // 同時に複数のbitが1であることがありうる。 - enum Directions: uint8_t { + enum Directions : uint8_t { DIRECTIONS_ZERO = 0, DIRECTIONS_RU = 1, DIRECTIONS_R = 2, DIRECTIONS_RD = 4, DIRECTIONS_U = 8, DIRECTIONS_D = 16, DIRECTIONS_LU = 32, DIRECTIONS_L = 64, DIRECTIONS_LD = 128, DIRECTIONS_CROSS = DIRECTIONS_U | DIRECTIONS_D | DIRECTIONS_R | DIRECTIONS_L, @@ -424,7 +424,7 @@ enum Bound { // -------------------- // 置換表に格納するときにあまりbit数が多いともったいないので値自体は16bitで収まる範囲で。 -enum Value: int32_t +enum Value : int32_t { VALUE_ZERO = 0, @@ -467,10 +467,10 @@ enum Value: int32_t }; // ply手で詰ませるときのスコア -constexpr Value mate_in(int ply) { return (Value)(VALUE_MATE - ply);} +constexpr Value mate_in(int ply) { return (Value)(VALUE_MATE - ply); } // ply手で詰まされるときのスコア -constexpr Value mated_in(int ply) { return (Value)(-VALUE_MATE + ply);} +constexpr Value mated_in(int ply) { return (Value)(-VALUE_MATE + ply); } // -------------------- @@ -638,7 +638,7 @@ struct Move16; // move = move16 + (piece << 16) // なので、Moveが使うのは、16bit(Move16) + 5bit(Piece) = 下位21bit // -enum Move: uint32_t { +enum Move : uint32_t { MOVE_NONE = 0, // 無効な移動 @@ -660,11 +660,11 @@ std::string to_usi_string(Move16 m); // それらを明確に区別したい時に用いる。 struct Move16 { - Move16():move(0){} - Move16(u16 m): move(m) {} + Move16() : move(0) {} + Move16(u16 m) : move(m) {} // Moveからの暗黙変換はできないとMOVE_NONEの代入などで困る。 - Move16(Move m) :move((u16)m){} + Move16(Move m) : move((u16)m) {} // uint16_tのまま取り出す。 // Moveに変換が必要なときは、そのあとMove()にcastすることはできる。(上位16bitは0のまま) @@ -1132,7 +1132,7 @@ namespace Eval // 　1) 16bitだと32bitだと思いこんでいてオーバーフローさせてしまうコードを書いてしまうことが多々あり、保守が困難。 // 　2) ここが32bitであってもそんなに速度低下しないし、それはSSE4.2以前に限るから許容範囲。 // という2つの理由から、32bitに固定する。 - enum BonaPiece: int32_t; + enum BonaPiece : int32_t; // 評価関数本体。 // 戻り値は、 From 8587c24de3b655fb8fdc9f1d8bb09d676b7059a7 Mon Sep 17 00:00:00 2001 From: KazApps Date: Fri, 9 Aug 2024 21:24:12 +0900 Subject: [PATCH 3/3] =?UTF-8?q?FOUND:;=E3=81=AE=E3=82=BB=E3=83=9F=E3=82=B3?= =?UTF-8?q?=E3=83=AD=E3=83=B3=E3=82=92=E3=82=82=E3=81=A8=E3=81=AB=E6=88=BB?= =?UTF-8?q?=E3=81=97=E3=81=9F?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- source/types.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/source/types.cpp b/source/types.cpp index 67fdd0d85..f7952fd6b 100644 --- a/source/types.cpp +++ b/source/types.cpp @@ -139,7 +139,7 @@ namespace Search { pos.undo_move(pv[0]); return false; - FOUND: + FOUND:; pos.undo_move(pv[0]); pv.push_back(m); // std::cout << m << std::endl;