Set up Windows workflow (#12)

* Used C++14 to compile the code
* Fixed Windows compile error on C++20 caused by dropping `const`
  * Which means I could leave the standard at C++20.
  * However, C++14 is reasonably recent, while still old enough that most compilers will support it.
* Added Windows package workflow using cibuildwheel
* Added back macOS 13 UTs
  * Although I package only on macOS 14 (because Hatch produces universal wheels).
  * Nonetheless, I should still test on the older architecture.

Author: tfpf

Diff for: .github/workflows/package.yml

@@ -26,12 +26,12 @@ jobs:
         name: wheel-${{ matrix.os }}-${{ matrix.python }}
         path: dist/*.whl
 
-  # Hatch builds wheels with a platform tag not permitted by PyPI. Hence, use
-  # cibuildwheel.
-  build_linux:
+  # Hatch builds Linux wheels with a platform tag not permitted by PyPI. Hence,
+  # use cibuildwheel.
+  build_ubuntu_windows:
     strategy:
       matrix:
-        os: ['ubuntu-22.04']
+        os: ['ubuntu-22.04', 'windows-2022']
     name: build on ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     steps:
@@ -45,7 +45,7 @@ jobs:
 
   release:
     if: github.ref_type == 'tag'
-    needs: [build_macos, build_linux]
+    needs: [build_macos, build_ubuntu_windows]
     runs-on: ubuntu-22.04
     permissions: write-all
     steps:

Diff for: .github/workflows/test.yml

@@ -5,7 +5,7 @@ jobs:
   test:
     strategy:
       matrix:
-        os: ['macos-14', 'ubuntu-22.04']
+        os: ['macos-13', 'macos-14', 'ubuntu-22.04', 'windows-2022']
         python: ['3.10', '3.11', '3.12', '3.13']
     name: test on ${{ matrix.os }} with ${{ matrix.python }}
     runs-on: ${{ matrix.os }}

Diff for: setup.py

@@ -1,10 +1,12 @@
+import platform
+
 import setuptools
 
 setuptools.setup(
     ext_modules=[
         setuptools.Extension(
             name="pysorteddict",
-            extra_compile_args=["-std=c++20"],
+            extra_compile_args=["/std:c++14" if platform.system() == "Windows" else "-std=c++14"],
             sources=["src/pysorteddict/pysorteddict.cc"],
             py_limited_api=True,
         )

Diff for: src/pysorteddict/pysorteddict.cc

@@ -75,9 +75,13 @@ static PyObject* sorted_dict_type_new(PyTypeObject* type, PyObject* args, PyObje
     }
 
     SortedDictType* sd = (SortedDictType*)self;
-    // Casting a string constant to a non-const pointer is not permitted in
-    // C++, but the signature of this function is such that I am forced to.
-    char* args_names[] = { "key_type", nullptr };
+    // Up to Python 3.12, the argument parser below took an array of pointers
+    // (with each pointer pointing to a C string) as its fourth argument.
+    // However, C++ does not allow converting a string constant to a pointer.
+    // Hence, I use a character array to construct the C string, and then place
+    // it in an array of pointers.
+    char key_type[] = "key_type";
+    char* args_names[] = { key_type, nullptr };
     if (!PyArg_ParseTupleAndKeywords(args, kwargs, "O|", args_names, &sd->key_type))
     {
         Py_DECREF(self);