Extract BBIO_err into a separate header

Author: cocasema

setup.py

@@ -33,6 +33,12 @@
                'Topic :: Home Automation',
                'Topic :: System :: Hardware']
 
+extension_args = {
+    'include_dirs': ['source/include/'],
+    'extra_compile_args': CFLAGS,
+    'define_macros': kernel41
+}
+
 setup(name             = 'Adafruit_BBIO',
       version          = '1.0.3',
       author           = 'Justin Cooper',
@@ -45,9 +51,9 @@
       classifiers      = classifiers,
       packages         = find_packages(),
       py_modules       = ['Adafruit_I2C'],
-      ext_modules      = [Extension('Adafruit_BBIO.GPIO', ['source/py_gpio.c', 'source/event_gpio.c', 'source/c_pinmux.c', 'source/constants.c', 'source/common.c'], extra_compile_args=CFLAGS, define_macros=kernel41),
-                          Extension('Adafruit_BBIO.PWM', ['source/py_pwm.c', 'source/c_pwm.c', 'source/c_pinmux.c', 'source/constants.c', 'source/common.c'], extra_compile_args=CFLAGS, define_macros=kernel41),
-                          Extension('Adafruit_BBIO.ADC', ['source/py_adc.c', 'source/c_adc.c', 'source/constants.c', 'source/common.c'], extra_compile_args=CFLAGS, define_macros=kernel41),
-                          Extension('Adafruit_BBIO.SPI', ['source/spimodule.c', 'source/constants.c', 'source/common.c'], extra_compile_args=CFLAGS, define_macros=kernel41),
-                          Extension('Adafruit_BBIO.UART', ['source/py_uart.c', 'source/c_uart.c', 'source/constants.c', 'source/common.c'], extra_compile_args=CFLAGS, define_macros=kernel41)] )
+      ext_modules      = [Extension('Adafruit_BBIO.GPIO', ['source/py_gpio.c', 'source/event_gpio.c', 'source/c_pinmux.c', 'source/constants.c', 'source/common.c'], **extension_args),
+                          Extension('Adafruit_BBIO.PWM', ['source/py_pwm.c', 'source/c_pwm.c', 'source/c_pinmux.c', 'source/constants.c', 'source/common.c'], **extension_args),
+                          Extension('Adafruit_BBIO.ADC', ['source/py_adc.c', 'source/c_adc.c', 'source/constants.c', 'source/common.c'], **extension_args),
+                          Extension('Adafruit_BBIO.SPI', ['source/spimodule.c', 'source/constants.c', 'source/common.c'], **extension_args),
+                          Extension('Adafruit_BBIO.UART', ['source/py_uart.c', 'source/c_uart.c', 'source/constants.c', 'source/common.c'], **extension_args)] )
 

source/common.h

@@ -33,6 +33,8 @@ SOFTWARE.
 
 #include <stddef.h>
 
+#include "adafruit/bbio/error.h"
+
 #define MODE_UNKNOWN -1
 #define BOARD        10
 #define BCM          11
@@ -42,16 +44,6 @@ SOFTWARE.
 #define FILENAME_BUFFER_SIZE 128
 #define MAX_PATH 256
 
-typedef enum {
-	BBIO_OK, // No error
-	BBIO_ACCESS, // Error accessing a file
-	BBIO_SYSFS, // Some error with Sysfs files
-	BBIO_CAPE, // some error with capes
-	BBIO_INVARG, // Invalid argument
-	BBIO_MEM,
-	BBIO_GEN // General error
-} BBIO_err;
-
 // Modeled after "pwm": submap in bone.js from bonescript
 // https://github.com/jadonk/bonescript/blob/master/src/bone.js#L680
 typedef struct pwm_t {

source/include/adafruit/bbio/error.h

@@ -0,0 +1,43 @@
+/*
+Copyright (c) 2017 Adafruit
+Copyright (c) 2017 Nikolay Semenov
+
+Permission is hereby granted, free of charge, to any person obtaining a copy of
+this software and associated documentation files (the "Software"), to deal in
+the Software without restriction, including without limitation the rights to
+use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
+of the Software, and to permit persons to whom the Software is furnished to do
+so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+*/
+#pragma once
+
+#ifdef __cplusplus
+namespace adafruit {
+namespace bbio {
+#endif
+
+typedef enum {
+	BBIO_OK, // No error
+	BBIO_ACCESS, // Error accessing a file
+	BBIO_SYSFS, // Some error with Sysfs files
+	BBIO_CAPE, // some error with capes
+	BBIO_INVARG, // Invalid argument
+	BBIO_MEM,
+	BBIO_GEN // General error
+} BBIO_err;
+
+#ifdef __cplusplus
+} // namespace bbio
+} // namespace adafruit
+#endif