[CLIENT-2527] CI/CD: compile Python client with -Werror (#496)

* Fix all warning errors

.github/workflows/tests.yml

@@ -57,6 +57,8 @@ jobs:
 
     - name: Build client
       run: python3 -m build
+      env:
+        CFLAGS: '-Werror'
 
     - name: Send wheel to test jobs
       uses: actions/upload-artifact@v3

VERSION

@@ -1 +1 @@
-14.0.0-rc.2
+14.0.0-rc.4

setup.py

@@ -91,6 +91,10 @@
     'z'
 ]
 
+##########################
+# GITHUB ACTIONS SETTINGS
+##########################
+
 if COVERAGE:
     extra_compile_args.append('-fprofile-arcs')
     extra_compile_args.append('-ftest-coverage')

src/main/aerospike.c

@@ -138,7 +138,7 @@ static int Aerospike_Clear(PyObject *aerospike)
 PyMODINIT_FUNC PyInit_aerospike(void)
 {
 
-    const char version[] = "14.0.0-rc.2";
+    const char version[] = "14.0.0-rc.4";
     // Makes things "thread-safe"
     Py_Initialize();
     int i = 0;
@@ -155,7 +155,7 @@ PyMODINIT_FUNC PyInit_aerospike(void)
     PyObject *aerospike = PyModule_Create(&moduledef);
 
     // In case adding objects to module fails, we can properly deallocate the module state later
-    memset(Aerospike_State(aerospike), NULL, sizeof(struct Aerospike_State));
+    memset(Aerospike_State(aerospike), 0, sizeof(struct Aerospike_State));
 
     Aerospike_Enable_Default_Logging();
 

src/main/client/bit_operate.c

@@ -396,7 +396,7 @@ static as_status add_op_bit_add(AerospikeClient *self, as_error *err, char *bin,
         return err->code;
     }
 
-    uint64_t action_int64;
+    int64_t action_int64;
     if (get_int64_t(err, ACTION_KEY, op_dict, &action_int64) != AEROSPIKE_OK) {
         return err->code;
     }
@@ -808,7 +808,7 @@ static as_status add_op_bit_subtract(AerospikeClient *self, as_error *err,
         return err->code;
     }
 
-    uint64_t action_int64;
+    int64_t action_int64;
     if (get_int64_t(err, ACTION_KEY, op_dict, &action_int64) != AEROSPIKE_OK) {
         return err->code;
     }

src/main/conversions.c

@@ -657,7 +657,6 @@ as_status pyobject_to_strArray(as_error *err, PyObject *py_list, char **arr,
 {
 
     as_error_reset(err);
-    PyObject *py_u_str = NULL;
 
     if (!PyList_Check(py_list)) {
         return as_error_update(err, AEROSPIKE_ERR_CLIENT, "not a list");
@@ -834,7 +833,7 @@ as_status pyobject_to_val(AerospikeClient *self, as_error *err,
         Py_DECREF(py_parameter);
 
         PyObject *geospatial_dump = AerospikeGeospatial_DoDumps(py_data, err);
-        char *geo_value = PyUnicode_AsUTF8(geospatial_dump);
+        const char *geo_value = PyUnicode_AsUTF8(geospatial_dump);
         char *geo_value_cpy = strdup(geo_value);
 
         Py_DECREF(py_data);
@@ -919,7 +918,7 @@ as_status pyobject_to_record(AerospikeClient *self, as_error *err,
         PyObject *key = NULL, *value = NULL;
         Py_ssize_t pos = 0;
         Py_ssize_t size = PyDict_Size(py_rec);
-        char *name = NULL;
+        const char *name;
         long ret_val = 0;
 
         as_record_init(rec, size);

src/main/query/where.c

@@ -354,7 +354,6 @@ AerospikeQuery *AerospikeQuery_Where_Invoke(AerospikeQuery *self,
 AerospikeQuery *AerospikeQuery_Where(AerospikeQuery *self, PyObject *args)
 {
     as_error err;
-    PyObject *py_err = NULL;
 
     PyObject *py_pred = NULL;
     PyObject *py_cdt_ctx = NULL;