Merge branch '2.x'

.gitignore

@@ -3,3 +3,4 @@
 .php_cs.cache
 vendor/
 .phpunit.cache/
+tools/php-cs-fixer/vendor/

.php-cs-fixer.dist.php

@@ -0,0 +1,18 @@
+<?php
+
+use PhpCsFixer\Config;
+
+$header = file_exists(__DIR__.'/header.txt') ? file_get_contents(__DIR__.'/header.txt') : '';
+$finder = PhpCsFixer\Finder::create()
+    ->in(__DIR__ . '/src')
+    ->in(__DIR__ . '/tests')
+;
+return (new Config())
+
+    ->setRules([
+        '@Symfony' => true,
+        'array_syntax' => ['syntax' => 'short'],
+        'header_comment' => ['header' => $header],
+    ])
+    ->setFinder($finder)
+;

.php_cs.dist

@@ -1,14 +0,0 @@
-<?php
-
-$finder = PhpCsFixer\Finder::create()
-    ->in(__DIR__)
-    ->exclude('var')
-;
-
-return PhpCsFixer\Config::create()
-    ->setRules([
-        '@Symfony' => true,
-        'array_syntax' => ['syntax' => 'short'],
-    ])
-    ->setFinder($finder)
-;

composer.json

@@ -3,13 +3,11 @@
     "description": "Implements _getch and _ungetch for windows and linux using ffi",
     "type": "library",
     "require": {
-        "php": ">=7.4",
+        "php": "^8.2",
         "ext-ffi": "*"
     },
   "require-dev": {
-        "friendsofphp/php-cs-fixer": "^2.18",
+        "phpunit/phpunit": "^9.6"
-        "jetbrains/phpstorm-stubs": "dev-master",
-        "phpunit/phpunit": "^9.5"
     },
   "autoload": {
     "psr-4": {

functions.php

@@ -3,7 +3,7 @@
 declare(strict_types=1);
 
 /*
- * Copyright (c) 2020  https://rewiv.com sikofitt@gmail.com
+ * Copyright (c) 2020-2024  https://sikofitt.com sikofitt@gmail.com
  *
  * This Source Code Form is subject to the
  * terms of the Mozilla Public License, v. 2.0.
@@ -15,19 +15,15 @@ declare(strict_types=1);
 use Sikofitt\Console\Getch;
 
 if (!function_exists('getch')) {
-    function getch(string $linuxLibrary = null): int
+    function getch(?string $linuxLibrary = null): int
     {
-        $g = new Getch($linuxLibrary);
+        return (new Getch($linuxLibrary))->getch();
-
-        return $g->getch();
     }
 }
 
 if (!function_exists('ungetch')) {
-    function ungetch($char, string $linuxLibrary = null): int
+    function ungetch($char, ?string $linuxLibrary = null): int
     {
-        $g = new Getch($linuxLibrary);
+        return (new Getch($linuxLibrary))->ungetch($char);
-
-        return $g->ungetch($char);
     }
 }

header.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2020  https://rewiv.com sikofitt@gmail.com
+Copyright (c) 2020-2024  https://sikofitt.com sikofitt@gmail.com
 
 This Source Code Form is subject to the 
 terms of the Mozilla Public License, v. 2.0. 

phpunit.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.5/phpunit.xsd"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.6/phpunit.xsd"
          bootstrap="vendor/autoload.php"
          cacheResultFile=".phpunit.cache/test-results"
          executionOrder="depends,defects"

src/Console/Getch.php

@@ -3,7 +3,7 @@
 declare(strict_types=1);
 
 /*
- * Copyright (c) 2020  https://rewiv.com sikofitt@gmail.com
+ * Copyright (c) 2020-2024  https://sikofitt.com sikofitt@gmail.com
  *
  * This Source Code Form is subject to the
  * terms of the Mozilla Public License, v. 2.0.
@@ -14,9 +14,6 @@ declare(strict_types=1);
 
 namespace Sikofitt\Console;
 
-use FFI;
-use RuntimeException;
-
 final class Getch
 {
     // Special key codes
@@ -25,7 +22,7 @@ final class Getch
     public const KEY_E0 = 0;
     public const KEY_E1 = 224;
 
-    // Supported scan scodes.
+    // Supported scan codes.
     public const KEY_F1 = 59;
     public const KEY_F2 = 60;
     public const KEY_F3 = 61;
@@ -56,14 +53,14 @@ final class Getch
         int _ungetch(int c);
     DECLARATIONS;
 
-    private static ?FFI $ffi = null;
+    private static ?\FFI $ffi = null;
 
     public static function resetFFI(): void
     {
-        static::$ffi = null;
+        self::$ffi = null;
     }
 
-    public function __construct(string $linuxLibrary = null)
+    public function __construct(?string $linuxLibrary = null)
     {
         if (null === $linuxLibrary) {
             $linuxLibrary = self::LINUX_LIBRARY;
@@ -73,15 +70,15 @@ final class Getch
             $osFamily = PHP_OS_FAMILY;
             if ('Windows' === $osFamily) {
                 $declarations = self::DECLARATIONS.' int _kbhit();';
-                self::$ffi = FFI::cdef($declarations, self::WINDOWS_LIBRARY);
+                self::$ffi = \FFI::cdef($declarations, self::WINDOWS_LIBRARY);
             } elseif ('Linux' === $osFamily) {
                 if (!file_exists($linuxLibrary)) {
-                    throw new RuntimeException(sprintf('Could not find library file %s.', $linuxLibrary));
+                    throw new \RuntimeException(sprintf('Could not find library file %s.', $linuxLibrary));
                 }
                 $declarations = self::DECLARATIONS.' int cinPeek();';
-                self::$ffi = FFI::cdef($declarations, $linuxLibrary);
+                self::$ffi = \FFI::cdef($declarations, $linuxLibrary);
             } else {
-                throw new RuntimeException(sprintf('Sorry, %s is not supported yet.', $osFamily));
+                throw new \RuntimeException(sprintf('Sorry, %s is not supported yet.', $osFamily));
             }
         }
     }
@@ -89,9 +86,9 @@ final class Getch
     public function peek(): int
     {
         if (PHP_OS_FAMILY === 'Windows') {
-            if ($ffi->_kbhit()) {
+            if (self::$ffi->_kbhit()) {
-                $result = $ffi->_getch();
+                $result = self::$ffi->_getch();
-                $ffi->_ungetch($result);
+                self::$ffi->_ungetch($result);
 
                 return $result;
             }
@@ -99,7 +96,7 @@ final class Getch
             return -1;
         }
 
-        return $ffi->cinPeek();
+        return self::$ffi->cinPeek();
     }
 
     public function getch(): int
@@ -107,12 +104,8 @@ final class Getch
         return self::$ffi->_getch();
     }
 
-    public function ungetch($char): int
+    public function ungetch(string|int $char): int
     {
-        if (!is_string($char) && !is_int($char)) {
-            throw new \TypeError('ungetch takes a parameter of int or string.');
-        }
-
         if (is_string($char)) {
             $char = ord($char[0]);
         }

tests/Getch/GetchTest.php

@@ -1,5 +1,15 @@
 <?php
 
+/*
+ * Copyright (c) 2020-2024  https://sikofitt.com sikofitt@gmail.com
+ *
+ * This Source Code Form is subject to the
+ * terms of the Mozilla Public License, v. 2.0.
+ *
+ * If a copy of the MPL was not distributed with this file,
+ * You can obtain one at https://mozilla.org/MPL/2.0/.
+ */
+
 namespace Sikofitt\Tests\Console\Getch;
 
 use PHPUnit\Framework\TestCase;
@@ -23,7 +33,9 @@ class GetchTest extends TestCase
 
     /**
      * @preserveGlobalState disabled
+     *
      * @backupStaticAttributes false
+     *
      * @backupGlobals false
      */
     public function testFailureOnInvalidLibrary()

tests/Ungetch/UngetchTest.php

@@ -1,5 +1,15 @@
 <?php
 
+/*
+ * Copyright (c) 2020-2024  https://sikofitt.com sikofitt@gmail.com
+ *
+ * This Source Code Form is subject to the
+ * terms of the Mozilla Public License, v. 2.0.
+ *
+ * If a copy of the MPL was not distributed with this file,
+ * You can obtain one at https://mozilla.org/MPL/2.0/.
+ */
+
 namespace Sikofitt\Tests\Console\Ungetch;
 
 use PHPUnit\Framework\TestCase;

tools/php-cs-fixer/composer.json

@@ -0,0 +1,5 @@
+{
+    "require-dev": {
+        "friendsofphp/php-cs-fixer": "^3.59"
+    }
+}