chess 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: ba8b98eeeaf058981ec4d1e12373b95b8d37babc
-  data.tar.gz: 592981be2deed30ca9f1d0fee03e22f0e04425de
+  metadata.gz: 94e7a4ac7ec5ae63ad5987a9fb74e1dbc5cb7800
+  data.tar.gz: bd70ea660bd6c47e95da157e5ea2a6024465f3ba
 SHA512:
-  metadata.gz: c43dd4b27bf44c9eb9c319abb130c93adfa7faa58303cf6b7275830d7d2cb164bef9a7da82adc10cb29bebafa14065acff55a74fc9f5bea712dd4d932e339b1e
-  data.tar.gz: d6ef9971478bbd5a0fa9c19cfe15923dc5a62f76fc1e82eaf06fdaf39d9b7e14d7fad5473a1703ff3f75f7f3d28ac3173c151d91b9844285cc028e6642a106b6
+  metadata.gz: e097138279b50d5465e58c47cc2ca5b1967c1872c3db54811efd8a3267874996c68eb87637b6cc1842ac5af94f027c2ca016b9865541ca06880255ed3db432ea
+  data.tar.gz: b14cb7c634fb759d2d485f653915a311c98aa756494c2f6fb5fa072879e0a641151b4c2e090e91222ce1fb18eac490f0f5d7e4bf33fc13683559336312b5caee

data/doc/Chess/Board.html

@@ -185,14 +185,12 @@ board_get_piece (VALUE self, VALUE square)
   Board *board;
   Data_Get_Struct (self, Board, board);
   int i;
-  char piece[2];
-  piece[1] = '\0';
   if (TYPE (square) == T_STRING)
     i = coord_to_square (StringValuePtr (square));
   else
     i = FIX2INT (square);
-  piece[0] = board->placement[i];
-  return rb_str_new2 (piece);
+  char piece = board->placement[i];
+  return rb_str_new (&piece, 1);
 }</pre>
           </div>
           
@@ -529,25 +527,22 @@ board_placement (VALUE self)
 {
   Board *board;
   Data_Get_Struct (self, Board, board);
-  int i;
-  char piece[2];
-  piece[1] = &#39;\0&#39;;
   if (!rb_block_given_p ())
     {
       VALUE placement = rb_ary_new ();
-      for (i = 0; i &lt; 64; i++)
+      for (int i = 0; i &lt; 64; i++)
         {
-          piece[0] = board-&gt;placement[i];
-          rb_ary_push (placement, rb_str_new2 (piece));
+          char piece = board-&gt;placement[i];
+          rb_ary_push (placement, rb_str_new (&amp;piece, 1));
         }
       return placement;
     }
   else
     {
-      for (i = 0; i &lt; 64; i++)
+      for (int i = 0; i &lt; 64; i++)
         {
-          piece[0] = board-&gt;placement[i];
-          rb_yield_values (2, rb_str_new2 (piece), INT2FIX (i));
+          char piece = board-&gt;placement[i];
+          rb_yield_values (2, rb_str_new (&amp;piece, 1), INT2FIX (i));
         }
       return self;
     }

data/doc/Chess/CGame.html

@@ -242,9 +242,8 @@ game_coord_moves (VALUE self)
 {
   Game *g;
   Data_Get_Struct (self, Game, g);
-  int i;
   VALUE moves = rb_ary_new ();
-  for (i = 0; i < g->current; i++)
+  for (int i = 0; i < g->current; i++)
     rb_ary_push (moves, rb_str_new2 (g->coord_moves[i]));
   return moves;
 }</pre>
@@ -372,12 +371,13 @@ game_each (VALUE self)
 {
   if (!rb_block_given_p ())
     return game_moves(self);
-  int i;
   Game *g;
   Data_Get_Struct (self, Game, g);
-  for (i = 0; i &lt; g-&gt;current; i++)
-    rb_yield_values (4, Data_Wrap_Struct (board_klass, 0, 0, get_board (g, i)),
-                     rb_str_new2 (g-&gt;moves[i]), rb_str_new2 (g-&gt;coord_moves[i]),
+  for (int i = 0; i &lt; g-&gt;current; i++)
+    rb_yield_values (4,
+                     Data_Wrap_Struct (board_klass, 0, 0, get_board (g, i)),
+                     rb_str_new2 (g-&gt;moves[i]),
+                     rb_str_new2 (g-&gt;coord_moves[i]),
                      INT2FIX (i));
   return self;
 }</pre>
@@ -652,9 +652,8 @@ game_moves (VALUE self)
 {
   Game *g;
   Data_Get_Struct (self, Game, g);
-  int i;
   VALUE moves = rb_ary_new ();
-  for (i = 0; i &lt; g-&gt;current; i++)
+  for (int i = 0; i &lt; g-&gt;current; i++)
     rb_ary_push (moves, rb_str_new2 (g-&gt;moves[i]));
   return moves;
 }</pre>

data/doc/README_rdoc.html

@@ -127,7 +127,7 @@ project and send a pull request.</p>
 
 <h2 id="label-Copyright">Copyright<span><a href="#label-Copyright">&para;</a> <a href="#top">&uarr;</a></span></h2>
 
-<p>Copyright © 2015 <a href="https://github.com/pioz">Enrico Pilotto
+<p>Copyright © 2017 <a href="https://github.com/pioz">Enrico Pilotto
 (@pioz)</a>. See <a
 href="https://github.com/pioz/chess/blob/master/LICENSE">LICENSE</a> for
 details.</p>

data/doc/created.rid

@@ -1,15 +1,15 @@
-Wed, 19 Jul 2017 17:57:58 +0200
-README.rdoc	Wed, 19 Jul 2017 16:55:54 +0200
-lib/chess.rb	Wed, 19 Jul 2017 16:55:54 +0200
-lib/chess/exceptions.rb	Wed, 19 Jul 2017 16:55:54 +0200
-lib/chess/game.rb	Wed, 19 Jul 2017 17:27:50 +0200
-lib/chess/gnuchess.rb	Wed, 19 Jul 2017 16:55:54 +0200
-lib/chess/pgn.rb	Wed, 19 Jul 2017 17:38:47 +0200
-lib/chess/utf8_notation.rb	Wed, 19 Jul 2017 16:55:54 +0200
-lib/chess/version.rb	Wed, 19 Jul 2017 16:55:54 +0200
-ext/bitboard.c	Wed, 19 Jul 2017 16:55:54 +0200
-ext/board.c	Wed, 19 Jul 2017 16:55:54 +0200
-ext/chess.c	Wed, 19 Jul 2017 17:57:47 +0200
-ext/common.c	Wed, 19 Jul 2017 16:55:54 +0200
-ext/game.c	Wed, 19 Jul 2017 17:57:08 +0200
-ext/special.c	Wed, 19 Jul 2017 16:55:54 +0200
+Thu, 20 Jul 2017 15:24:00 +0200
+README.rdoc	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/exceptions.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/game.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/gnuchess.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/pgn.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/utf8_notation.rb	Thu, 20 Jul 2017 14:48:45 +0200
+lib/chess/version.rb	Thu, 20 Jul 2017 14:48:45 +0200
+ext/bitboard.c	Thu, 20 Jul 2017 15:18:42 +0200
+ext/board.c	Thu, 20 Jul 2017 15:21:27 +0200
+ext/chess.c	Thu, 20 Jul 2017 15:14:18 +0200
+ext/common.c	Thu, 20 Jul 2017 14:48:45 +0200
+ext/game.c	Thu, 20 Jul 2017 15:22:45 +0200
+ext/special.c	Thu, 20 Jul 2017 14:48:45 +0200

data/doc/index.html

@@ -144,7 +144,7 @@ project and send a pull request.</p>
 
 <h2 id="label-Copyright">Copyright<span><a href="#label-Copyright">&para;</a> <a href="#top">&uarr;</a></span></h2>
 
-<p>Copyright © 2015 <a href="https://github.com/pioz">Enrico Pilotto
+<p>Copyright © 2017 <a href="https://github.com/pioz">Enrico Pilotto
 (@pioz)</a>. See <a
 href="https://github.com/pioz/chess/blob/master/LICENSE">LICENSE</a> for
 details.</p>

data/ext/bitboard.c

@@ -94,9 +94,8 @@ square2 (bboard b)
 void
 squares (bboard b, int *array, int *n)
 {
-  int i;
   *n = 0;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     if (b & 1ULL << i)
       {
         array[*n] = i;
@@ -130,10 +129,9 @@ get2 (bboard b, int file, int rank)
 void
 print_bitboard (bboard b)
 {
-  int i, j;
-  for (i = 7; i >= 0; i--) // rank => top to bottom
+  for (int i = 7; i >= 0; i--) // rank => top to bottom
     {
-      for (j = 0; j < 8; j++) // file => left to right
+      for (int j = 0; j < 8; j++) // file => left to right
         printf ("%d ", get2 (b, j, i) ? 1 : 0);
       printf ("\n");
     }
@@ -228,8 +226,7 @@ sliding_attacks (bboard slider, bboard propagator, int dir)
 void
 precalculate_xray_attack_white_pawn (bboard xray[64])
 {
-  int i;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
       xray[i] = EMPTY_BOARD;
       xray[i] = shift_one (1ULL << i, 7) | shift_one (1ULL << i, 1);
@@ -239,8 +236,7 @@ precalculate_xray_attack_white_pawn (bboard xray[64])
 void
 precalculate_xray_attack_black_pawn (bboard xray[64])
 {
-  int i;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
       xray[i] = EMPTY_BOARD;
       xray[i] = shift_one (1ULL << i, 5) | shift_one (1ULL << i, 3);
@@ -250,8 +246,7 @@ precalculate_xray_attack_black_pawn (bboard xray[64])
 void
 precalculate_xray_knight (bboard xray[64])
 {
-  int i;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
       xray[i] = 1ULL << i;
       xray[i] = ((xray[i] << 17) & NOT_FILE_A)
@@ -268,11 +263,10 @@ precalculate_xray_knight (bboard xray[64])
 void
 precalculate_xray_king (bboard xray[64])
 {
-  int i, j;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
       xray[i] = EMPTY_BOARD;
-      for (j = 0; j < 8; j++)
+      for (int j = 0; j < 8; j++)
         xray[i] = sliding_attacks (1ULL << i, EMPTY_BOARD, j) | xray[i];
     }
 }
@@ -328,10 +322,9 @@ xray_attack_black_pawn (int square)
 bboard
 xray_rook (bboard occupied_square, int square)
 {
-  int i, dir;
   bboard xray[4];
   bboard collision, shielded_square;
-  for (i = 0, dir = 0; i < 4; i++, dir+=2)
+  for (int i = 0, dir = 0; i < 4; i++, dir+=2)
     {
       xray[i] = sliding_attacks (1ULL << square, FULL_BOARD, dir);
       collision = xray[i] & occupied_square;
@@ -350,10 +343,9 @@ xray_knight (int square)
 bboard
 xray_bishop (bboard occupied_square, int square)
 {
-  int i, dir;
   bboard xray[4];
   bboard collision, shielded_square;
-  for (i = 0, dir = 1; i < 4; i++, dir+=2)
+  for (int i = 0, dir = 1; i < 4; i++, dir+=2)
     {
       xray[i] = sliding_attacks (1ULL << square, FULL_BOARD, dir);
       collision = xray[i] & occupied_square;

data/ext/board.c

@@ -55,12 +55,11 @@ print_board (Board *board)
 {
   char *s = (char *) malloc (251);
   int si = 0;
-  int i, j;
-  for (i = 7; i >= 0; i--) // rank => top to bottom
+  for (int i = 7; i >= 0; i--) // rank => top to bottom
     {
       sprintf (&s[si], "\e[37m%d\e[0m ", i + 1);
       si += 11;
-      for (j = 0; j < 8; j++) // file => left to right
+      for (int j = 0; j < 8; j++) // file => left to right
         {
           char piece = board->placement[8 * i + j];
           sprintf (&s[si], "%c ", piece == 0 ? '.' : piece);
@@ -145,11 +144,10 @@ xray (Board *board, int from, bool only_attack)
 bboard
 all_xray (Board *board, int color, bool only_attack)
 {
-  int i, piece_color;
   bboard x = EMPTY_BOARD; // xray for all pieces
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
-      piece_color = get_color (board, i);
+      int piece_color = get_color (board, i);
       // Calculate xray for pieces of color [c]
       if (piece_color == color)
         x |= xray (board, i, only_attack);
@@ -180,10 +178,10 @@ remove_piece (Board *board, int square, Board *new_board)
 int
 same_pieces_that_can_capture_a_square (Board *board, int color, int square, int *pieces, char piece_filter)
 {
-  int i, index = 0;
+  int index = 0;
   char p;
   Board new_board;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     if (get_color (board, i) == color)
       {
         p = board->placement[i];
@@ -268,11 +266,10 @@ king_in_checkmate (Board *board, int color)
 bool
 stalemate (Board *board, int color)
 {
-  int i, j;
   int s[64];
   int n;
   Board new_board;
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     if (get_color (board, i) == color)
       {
         bboard b = xray (board, i, FALSE) & ~board->pieces[color];
@@ -280,7 +277,7 @@ stalemate (Board *board, int color)
         if (b)
           {
             squares (b, s, &n);
-            for (j = 0; j < n; j++)
+            for (int j = 0; j < n; j++)
               if (try_move (board, i, s[j], 'Q', &new_board, 0, 0))
                 return FALSE;
           }
@@ -342,15 +339,15 @@ pseudo_legal_move (Board *board, int from, int to)
 void
 get_coord (Board *board, char piece, const char *disambiguating, const char *to_coord, char promote_in, int *from, int *to)
 {
-  int i, c, count = 0;
+  int count = 0;
   char file, rank;
   bboard x;
   *to = coord_to_square (to_coord);
   if (!piece)
     piece = 'P';
-  for (i = 0; i < 64; i++)
+  for (int i = 0; i < 64; i++)
     {
-      c = get_color (board, i);
+      int c = get_color (board, i);
       if (c == board->active_color && piece == toupper (board->placement[i]))
         {
           x = xray (board, i, FALSE) & ~board->pieces[c];
@@ -514,11 +511,11 @@ to_fen (Board *board)
 {
   // 1. Placement
   char placement[65];
-  int i, j, cur = 0;
+  int cur = 0;
   char p, pp = '-';
-  for (i = 7; i >= 0; i--)
+  for (int i = 7; i >= 0; i--)
     {
-      for (j = 0; j < 8; j++)
+      for (int j = 0; j < 8; j++)
         {
           p = board->placement[j+i*8];
           if (p == '\0' && p == pp)

data/ext/chess.c

@@ -248,9 +248,8 @@ game_moves (VALUE self)
 {
   Game *g;
   Data_Get_Struct (self, Game, g);
-  int i;
   VALUE moves = rb_ary_new ();
-  for (i = 0; i < g->current; i++)
+  for (int i = 0; i < g->current; i++)
     rb_ary_push (moves, rb_str_new2 (g->moves[i]));
   return moves;
 }
@@ -265,9 +264,8 @@ game_coord_moves (VALUE self)
 {
   Game *g;
   Data_Get_Struct (self, Game, g);
-  int i;
   VALUE moves = rb_ary_new ();
-  for (i = 0; i < g->current; i++)
+  for (int i = 0; i < g->current; i++)
     rb_ary_push (moves, rb_str_new2 (g->coord_moves[i]));
   return moves;
 }
@@ -344,12 +342,13 @@ game_each (VALUE self)
 {
   if (!rb_block_given_p ())
     return game_moves(self);
-  int i;
   Game *g;
   Data_Get_Struct (self, Game, g);
-  for (i = 0; i < g->current; i++)
-    rb_yield_values (4, Data_Wrap_Struct (board_klass, 0, 0, get_board (g, i)),
-                     rb_str_new2 (g->moves[i]), rb_str_new2 (g->coord_moves[i]),
+  for (int i = 0; i < g->current; i++)
+    rb_yield_values (4,
+                     Data_Wrap_Struct (board_klass, 0, 0, get_board (g, i)),
+                     rb_str_new2 (g->moves[i]),
+                     rb_str_new2 (g->coord_moves[i]),
                      INT2FIX (i));
   return self;
 }
@@ -399,25 +398,22 @@ board_placement (VALUE self)
 {
   Board *board;
   Data_Get_Struct (self, Board, board);
-  int i;
-  char piece[2];
-  piece[1] = '\0';
   if (!rb_block_given_p ())
     {
       VALUE placement = rb_ary_new ();
-      for (i = 0; i < 64; i++)
+      for (int i = 0; i < 64; i++)
         {
-          piece[0] = board->placement[i];
-          rb_ary_push (placement, rb_str_new2 (piece));
+          char piece = board->placement[i];
+          rb_ary_push (placement, rb_str_new (&piece, 1));
         }
       return placement;
     }
   else
     {
-      for (i = 0; i < 64; i++)
+      for (int i = 0; i < 64; i++)
         {
-          piece[0] = board->placement[i];
-          rb_yield_values (2, rb_str_new2 (piece), INT2FIX (i));
+          char piece = board->placement[i];
+          rb_yield_values (2, rb_str_new (&piece, 1), INT2FIX (i));
         }
       return self;
     }
@@ -427,7 +423,7 @@ board_placement (VALUE self)
  * call-seq: [square]
  *
  * Returns the piece on the +square+ of the chessboard. If there is no piece or
- * the square is not valid return the blank string.
+ * the square is not valid return +nil+.
  *
  * Each square on the chessboard is represented by an integer according to the
  * following scheme:
@@ -444,7 +440,7 @@ board_placement (VALUE self)
  *
  * Parameters are:
  * +square+:: the square of the board. Can be a fixnum between 0 and 63 or a
- * string like 'a2', 'c5'... The string must be downcase.
+ * string like 'a2', 'c5'...
  */
 VALUE
 board_get_piece (VALUE self, VALUE square)
@@ -452,14 +448,15 @@ board_get_piece (VALUE self, VALUE square)
   Board *board;
   Data_Get_Struct (self, Board, board);
   int i;
-  char piece[2];
-  piece[1] = '\0';
   if (TYPE (square) == T_STRING)
     i = coord_to_square (StringValuePtr (square));
   else
     i = FIX2INT (square);
-  piece[0] = board->placement[i];
-  return rb_str_new2 (piece);
+  char piece = board->placement[i];
+  if (i < 0 || i > 63 || !piece)
+    return Qnil;
+  else
+    return rb_str_new (&piece, 1);
 }
 
 /*

data/ext/common.c

@@ -22,7 +22,7 @@ square_to_rank (int square)
 int
 coord_to_square (const char *coord)
 {
-  return 8 * (coord[1] - 49) + (coord[0] - 97);
+  return 8 * ((coord[1] | ' ') - 49) + ((coord[0] | ' ') - 97);
 }
 
 char*

data/ext/game.c

@@ -29,8 +29,7 @@ init_game ()
 void
 free_game (Game *g)
 {
-  int i;
-  for (i = 0; i < g->current; i++)
+  for (int i = 0; i < g->current; i++)
     {
       free (g->boards[i]);
       free (g->moves[i]);
@@ -54,7 +53,7 @@ get_board (Game *g, int index)
     return &STARTING_BOARD;
   if (index < g->current)
     return g->boards[index];
-  return 0;
+  return NULL;
 }
 
 char*
@@ -147,8 +146,8 @@ threefold_repetition (Game *g)
   char* s[g->current + 1];
   s[0] = (char *) malloc (80);
   strcpy (s[0], "rnbqkbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBQKBNR w KQkq -");
-  int i, j;
   bool found = FALSE;
+  int i, j;
   for (i = 0; i < g->current; i++)
     {
       fen = to_fen (g->boards[i]);
@@ -344,9 +343,9 @@ main ()
 {
   // Valgrind run
   init_chess_library ();
-  int i, from, to;
+  int from, to;
 
-  for (i = 0; i < 1000; i++)
+  for (int i = 0; i < 1000; i++)
     {
       Game *g = init_game ();
       Board *board;

data/lib/chess/version.rb

@@ -1,5 +1,5 @@
 # The Chess library module.
 module Chess
   # The library version.
-  VERSION = '0.1.1'
+  VERSION = '0.1.2'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chess
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Enrico Pilotto
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-07-19 00:00:00.000000000 Z
+date: 2017-08-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler