In case of encode/decode error return empty data type, not False.

Author: erikbos

README.md

@@ -58,11 +58,11 @@ git clone https://github.com/mapcode-foundation/mapcode-cpp
 cd mapcode-python
 ```
 
-Compile the package in current directory: `python setup.py --inplace`
+Compile the package in current directory: `python setup.py build_ext --inplace`
 Install in your Python environment using: `python setup.py install`
 
 
-# Usage in Python
+# Python methods
 
 ## Mapcode syntax validation
 
@@ -105,7 +105,7 @@ print mapcode.encode_single(52.376514, 4.908542)
 NLD 49.4V
 
 For this coordinate there are multiple mapcodes possible, hence providing
-a territory code can give a different one.
+a territory code can give a different result.
 
 print mapcode.encode_single(41.851944, 12.433114)
 ITA 0Z.0Z
@@ -133,7 +133,7 @@ print mapcode.encode(39.609999,45.949999, 'AZE')
 
 ## Decoding
 
-Use the decode() method to decode a mapcode back to latitude and longitude.
+Use the decode() method to translate a mapcode back to latitude and longitude.
 
 ```python
 print mapcode.decode('NLD 49.4V')
@@ -161,5 +161,5 @@ A territory is require to give context to decoding:
 print mapcode.decode('IN VY.HV','USA')
 (39.72795, -86.118444)
 print mapcode.decode('IN VY.HV','RUS')
-(43.193485, 44.826592)```
-```
+(43.193485, 44.826592)
+```

mapcodemodule.c

@@ -73,10 +73,12 @@ static PyObject *decode(PyObject *self, PyObject *args)
     } else
         territorycode = 0;
 
-    if (decodeMapcodeToLatLon(&latitude, &longitude, mapcode, territorycode))
-        return Py_False;
-
-    return Py_BuildValue("dd", latitude, longitude);
+    if (decodeMapcodeToLatLon(&latitude, &longitude, mapcode, territorycode)) {
+        /* return 999/999 as error values */
+        latitude = 999;
+        longitude = 999;
+    }
+    return Py_BuildValue("ff", latitude, longitude);
 }
 
 
@@ -103,7 +105,7 @@ static PyObject *encode(PyObject *self, PyObject *args)
     if (territoryname) {
         territorycode = convertTerritoryIsoNameToCode(territoryname, 0);
         if (territorycode < 0) {
-            return Py_False;
+            return PyList_New(0);
         }
     }
 
@@ -119,7 +121,7 @@ static PyObject *encode(PyObject *self, PyObject *args)
         }
         return result;
     }
-    return Py_False;
+    return PyList_New(0);
 }
 
 
@@ -144,7 +146,7 @@ static PyObject *encode_single(PyObject *self, PyObject *args)
     if (territoryname) {
         territorycode = convertTerritoryIsoNameToCode(territoryname, 0);
         if (territorycode < 0)
-            return Py_False;
+            return Py_BuildValue("s", NULL);
     }
 
     // printf("debug: encode_single: territorystring: %s, code: %d\n", territoryname, territorycode);
@@ -154,7 +156,7 @@ static PyObject *encode_single(PyObject *self, PyObject *args)
         result = Py_BuildValue("s", encode_single_result);
         return result;
     } else
-        return Py_False;
+        return Py_BuildValue("s", NULL);
 }