Add auto build uploading

Author: xia-mc

.github/workflows/python-package.yml

@@ -26,7 +26,7 @@ jobs:
         flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
 
   build:
-    name: Test
+    name: Build
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
@@ -60,6 +60,24 @@ jobs:
         else
           chmod +x ./build.sh && ./build.sh
         fi
+
+    - name: Find Correct Build
+      id: findbuild
+      shell: bash
+      run: |
+        if [[ "$RUNNER_OS" == "Windows" ]]; then
+          output=__pyfastutil.pyd
+        else
+          output=__pyfastutil.so
+        fi
+        echo "::set-output name=libname::$output"
+
+    - name: Upload Build Artifacts
+      uses: actions/upload-artifact@v3
+      with:
+        name: ${{ steps.findbuild.outputs.libname }}
+        path: pyfastutil/${{ steps.findbuild.outputs.libname }}
+        
     - name: Test with pytest and generate coverage report
       run: |
         pytest --cov=your_package_name --cov-report=xml

pyfastutil/src/ints/BigIntArrayList.cpp

@@ -412,9 +412,9 @@ static PyObject *BigIntArrayList_remove(PyObject *pySelf, PyObject *object) {
 static PyObject *BigIntArrayList_sort(PyObject *pySelf, PyObject *args, PyObject *kwargs) {
     auto *self = reinterpret_cast<BigIntArrayList *>(pySelf);
 
-    PyObject *keyFunc = nullptr;
+    PyObject *keyFunc = Py_None;
     int reverseInt = 0;  // default: false
-    static const char *kwlist[] = {"key", "reverse", nullptr};
+    static constexpr const char *kwlist[] = {"key", "reverse", nullptr};
 
     if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|Oi", const_cast<char **>(kwlist), &keyFunc, &reverseInt)) {
         return nullptr;
@@ -424,7 +424,7 @@ static PyObject *BigIntArrayList_sort(PyObject *pySelf, PyObject *args, PyObject
 
     // do sort
     try {
-        if (keyFunc == nullptr || keyFunc == Py_None) {
+        if (keyFunc == Py_None) {
             if (self->vector.size() >= 32) {
                 // simd sort with auto-fallback
                 Py_BEGIN_ALLOW_THREADS

pyfastutil/src/ints/IntArrayList.cpp

@@ -414,7 +414,7 @@ static PyObject *IntArrayList_sort(PyObject *pySelf, PyObject *args, PyObject *k
 
     PyObject *keyFunc = nullptr;
     int reverseInt = 0;  // default: false
-    static const char *kwlist[] = {"key", "reverse", nullptr};
+    static constexpr const char *kwlist[] = {"key", "reverse", nullptr};
 
     if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|Oi", const_cast<char **>(kwlist), &keyFunc, &reverseInt)) {
         return nullptr;