define ColorChar (#110)

* define ColorChar

* bump version

* enhance shell robust

* fix command for linux

* mild fix test

* use bash

* mild fix

* analyzer

* add comment

* mild fix comment

* specify shell

Author: sensuikan1973

.github/workflows/dart_ci.yaml

@@ -64,10 +64,11 @@ jobs:
     - uses: dart-lang/setup-dart@v1
 
     - name: build libedax
+      shell: bash
       env:
         dst: '.'
         libedax_build_command: ${{ matrix.libedax_build_command }}
-      run: sh scripts/build_libedax.sh
+      run: ./scripts/build_libedax.sh
 
     - name: output build info
       shell: bash

.github/workflows/publish_libedax_assets.yaml

@@ -26,10 +26,11 @@ jobs:
       - uses: actions/checkout@v2
 
       - name: build libedax
+        shell: bash
         env:
           dst: ${{ env.OUTPUT_DIR }}
           libedax_build_command: ${{ matrix.libedax_build_command }}
-        run: sh scripts/build_libedax.sh
+        run: ./scripts/build_libedax.sh
 
       - name: output build info
         shell: bash

CHANGELOG.md

@@ -1,3 +1,7 @@
+# 2.7.0
+
+- add const `ColorChar`.
+
 # 2.6.0
 
 - upgrade dependencies

example/pubspec.lock

@@ -84,7 +84,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "2.6.0"
+    version: "2.7.0"
   meta:
     dependency: transitive
     description:

example/pubspec.yaml

@@ -1,5 +1,5 @@
 name: libedax4dart_example
-version: 2.6.0-example
+version: 2.7.0-example
 publish_to: none
 
 environment:

lib/src/board.dart

@@ -58,8 +58,8 @@ class Board {
   ///
   /// e.g. `-------------------*-------**O----**O*-----O--------------------W`.
   String stringApplicableToSetboard(final int currentColor) {
-    final pStone = currentColor == TurnColor.black ? '*' : 'O';
-    final oStone = currentColor == TurnColor.black ? 'O' : '*';
+    final pStone = currentColor == TurnColor.black ? ColorChar.black : ColorChar.white;
+    final oStone = currentColor == TurnColor.black ? ColorChar.white : ColorChar.black;
 
     final buffer = StringBuffer();
     for (var k = 0; k < 8; k++) {
@@ -93,8 +93,8 @@ class Board {
   /// 8 - - - - - - - - 8 <br>
   ///   A B C D E F G H
   String prettyString(final int currentColor) {
-    final pStone = currentColor == TurnColor.black ? '*' : 'O';
-    final oStone = currentColor == TurnColor.black ? 'O' : '*';
+    final pStone = currentColor == TurnColor.black ? ColorChar.black : ColorChar.white;
+    final oStone = currentColor == TurnColor.black ? ColorChar.white : ColorChar.black;
 
     final buffer = StringBuffer()..writeln('  A B C D E F G H');
     for (var k = 0; k < 8; k++) {

lib/src/constants.dart

@@ -6,6 +6,15 @@ class TurnColor {
   static const white = 1;
 }
 
+/// See: https://github.com/abulmo/edax-reversi/blob/1ae7c9fe5322ac01975f1b3196e788b0d25c1e10/src/board.c#L144-L200
+class ColorChar {
+  /// BLACK char. You can use this for `LibEdax.edaxSetboard`.
+  static const black = '*';
+
+  /// WHITE char. You can use this for `LibEdax.edaxSetboard`.
+  static const white = 'O';
+}
+
 class MoveMark {
   /// PASS string of BLACK.
   ///

pubspec.yaml

@@ -1,7 +1,7 @@
 # See: https://dart.dev/tools/pub/publishing
 
 name: libedax4dart
-version: 2.6.0
+version: 2.7.0
 description: Dart wrapper for libedax. With using libedax4dart, you can execute functions equivalent to edax.
 homepage: https://github.com/sensuikan1973/libedax4dart
 documentation: https://sensuikan1973.github.io/libedax4dart

scripts/build_libedax.sh

@@ -1,18 +1,18 @@
+#!/bin/bash
+set -euxo pipefail
+
 # NOTE: require some environment variables.
 # libedax_build_command (e.g. make libbuild ARCH=x64-modern COMP=gcc OS=osx)
 # dst: (e.g. build)
 #
 # example:
 # libedax_build_command="make libbuild ARCH=x64-modern COMP=gcc OS=osx" dst="." ./scripts/build_libedax.sh
 
+rm -rf edax-reversi
 git clone https://github.com/sensuikan1973/edax-reversi
 cd edax-reversi
 git remote update --prune
-
-git checkout .
-
 git switch libedax_sensuikan1973
-git pull
 git checkout $(cat ../.libedax-version)
 
 mkdir -p data

test/edax_command_test.dart

@@ -83,14 +83,14 @@ void main() {
     });
 
     test('setBoard', () {
-      const boardString = '-W----W--------------------WB------WBB-----W--------------------B';
+      const boardString = '-O----O--------------------O*------O**-----O--------------------B';
       final edax = LibEdax()..libedaxInitialize();
       sleep(const Duration(seconds: 1));
       edax
         ..edaxInit()
         ..edaxSetboard(boardString);
-      expect(edax.edaxGetDisc(TurnColor.white), 'W'.allMatches(boardString).length);
-      expect(edax.edaxGetDisc(TurnColor.black), 'B'.allMatches(boardString).length - 1);
+      expect(edax.edaxGetDisc(TurnColor.white), ColorChar.white.allMatches(boardString).length);
+      expect(edax.edaxGetDisc(TurnColor.black), ColorChar.black.allMatches(boardString).length);
       expect(edax.edaxGetCurrentPlayer(), TurnColor.black);
       edax.libedaxTerminate();
     });