Reformat Python bindings and ice40 main

Signed-off-by: David Shah <davey1576@gmail.com>

2 files changed, 177 insertions, 173 deletions

diff --git a/common/pybindings.cc b/common/pybindings.cc
index c3818cad..5c86720e 100644
--- a/common/pybindings.cc
+++ b/common/pybindings.cc
@@ -18,134 +18,138 @@
  *
  */
 
-
-#include "design.h"
 #include "chip.h"
+#include "design.h"
 #include "emb.h"
 
 // include after design.h/chip.h
 #include "pybindings.h"
 
-// Required to determine concatenated module name (which differs for different archs)
-#define PASTER(x, y) x ## _ ## y
-#define EVALUATOR(x, y)  PASTER(x,y)
+// Required to determine concatenated module name (which differs for different
+// archs)
+#define PASTER(x, y) x##_##y
+#define EVALUATOR(x, y) PASTER(x, y)
 #define MODULE_NAME EVALUATOR(nextpnrpy, ARCHNAME)
 #define PYINIT_MODULE_NAME EVALUATOR(&PyInit_nextpnrpy, ARCHNAME)
 #define STRINGIFY(x) #x
 #define TOSTRING(x) STRINGIFY(x)
 
-// Architecture-specific bindings should be created in the below function, which must be implemented in all
-// architectures
+// Architecture-specific bindings should be created in the below function, which
+// must be implemented in all architectures
 void arch_wrap_python();
 
-bool operator==(const PortRef &a, const PortRef &b) {
-	return (a.cell == b.cell) && (a.port == b.port);
+bool operator==(const PortRef &a, const PortRef &b)
+{
+    return (a.cell == b.cell) && (a.port == b.port);
 }
 
-BOOST_PYTHON_MODULE (MODULE_NAME) {
-	class_<GraphicElement>("GraphicElement")
-			.def_readwrite("style", &GraphicElement::style)
-			.def_readwrite("type", &GraphicElement::type)
-			.def_readwrite("x1", &GraphicElement::x1)
-			.def_readwrite("y1", &GraphicElement::y1)
-			.def_readwrite("x2", &GraphicElement::x2)
-			.def_readwrite("y2", &GraphicElement::y2)
-			.def_readwrite("text", &GraphicElement::text);
-
-	class_<PortRef>("PortRef")
-			.def_readwrite("cell", &PortRef::cell)
-			.def_readwrite("port", &PortRef::port);
-
-	class_<NetInfo, NetInfo*>("NetInfo")
-			.def_readwrite("name", &NetInfo::name)
-			.def_readwrite("driver", &NetInfo::driver)
-			.def_readwrite("users", &NetInfo::users)
-			.def_readwrite("attrs", &NetInfo::attrs)
-			.def_readwrite("wires", &NetInfo::wires);
-
-	WRAP_MAP(decltype(NetInfo::attrs), "IdStrMap");
-
-	class_<vector<PortRef>>("PortRefVector")
-			.def(vector_indexing_suite<vector<PortRef>>());
-
-	enum_<PortType>("PortType")
-			.value("PORT_IN", PORT_IN)
-			.value("PORT_OUT", PORT_OUT)
-			.value("PORT_INOUT", PORT_INOUT)
-			.export_values();
-
-	class_<PortInfo>("PortInfo")
-			.def_readwrite("name", &PortInfo::name)
-			.def_readwrite("net", &PortInfo::net)
-			.def_readwrite("type", &PortInfo::type);
-
-	class_<CellInfo, CellInfo*>("CellInfo")
-			.def_readwrite("name", &CellInfo::name)
-			.def_readwrite("type", &CellInfo::type)
-			.def_readwrite("ports", &CellInfo::ports)
-			.def_readwrite("attrs", &CellInfo::attrs)
-			.def_readwrite("params", &CellInfo::params)
-			.def_readwrite("bel", &CellInfo::bel)
-			.def_readwrite("pins", &CellInfo::pins);
-
-	WRAP_MAP(decltype(CellInfo::ports), "IdPortMap");
-	//WRAP_MAP(decltype(CellInfo::pins), "IdIdMap");
-
-	class_<Design, Design*>("Design", no_init)
-			.def_readwrite("chip", &Design::chip)
-			.def_readwrite("nets", &Design::nets)
-			.def_readwrite("cells", &Design::cells);
-
-	WRAP_MAP(decltype(Design::nets), "IdNetMap");
-	WRAP_MAP(decltype(Design::cells), "IdCellMap");
-
-	arch_wrap_python();
+BOOST_PYTHON_MODULE(MODULE_NAME)
+{
+    class_<GraphicElement>("GraphicElement")
+            .def_readwrite("style", &GraphicElement::style)
+            .def_readwrite("type", &GraphicElement::type)
+            .def_readwrite("x1", &GraphicElement::x1)
+            .def_readwrite("y1", &GraphicElement::y1)
+            .def_readwrite("x2", &GraphicElement::x2)
+            .def_readwrite("y2", &GraphicElement::y2)
+            .def_readwrite("text", &GraphicElement::text);
+
+    class_<PortRef>("PortRef")
+            .def_readwrite("cell", &PortRef::cell)
+            .def_readwrite("port", &PortRef::port);
+
+    class_<NetInfo, NetInfo *>("NetInfo")
+            .def_readwrite("name", &NetInfo::name)
+            .def_readwrite("driver", &NetInfo::driver)
+            .def_readwrite("users", &NetInfo::users)
+            .def_readwrite("attrs", &NetInfo::attrs)
+            .def_readwrite("wires", &NetInfo::wires);
+
+    WRAP_MAP(decltype(NetInfo::attrs), "IdStrMap");
+
+    class_<vector<PortRef>>("PortRefVector")
+            .def(vector_indexing_suite<vector<PortRef>>());
+
+    enum_<PortType>("PortType")
+            .value("PORT_IN", PORT_IN)
+            .value("PORT_OUT", PORT_OUT)
+            .value("PORT_INOUT", PORT_INOUT)
+            .export_values();
+
+    class_<PortInfo>("PortInfo")
+            .def_readwrite("name", &PortInfo::name)
+            .def_readwrite("net", &PortInfo::net)
+            .def_readwrite("type", &PortInfo::type);
+
+    class_<CellInfo, CellInfo *>("CellInfo")
+            .def_readwrite("name", &CellInfo::name)
+            .def_readwrite("type", &CellInfo::type)
+            .def_readwrite("ports", &CellInfo::ports)
+            .def_readwrite("attrs", &CellInfo::attrs)
+            .def_readwrite("params", &CellInfo::params)
+            .def_readwrite("bel", &CellInfo::bel)
+            .def_readwrite("pins", &CellInfo::pins);
+
+    WRAP_MAP(decltype(CellInfo::ports), "IdPortMap");
+    // WRAP_MAP(decltype(CellInfo::pins), "IdIdMap");
+
+    class_<Design, Design *>("Design", no_init)
+            .def_readwrite("chip", &Design::chip)
+            .def_readwrite("nets", &Design::nets)
+            .def_readwrite("cells", &Design::cells);
+
+    WRAP_MAP(decltype(Design::nets), "IdNetMap");
+    WRAP_MAP(decltype(Design::cells), "IdCellMap");
+
+    arch_wrap_python();
 }
 
-void arch_appendinittab() {
-	PyImport_AppendInittab(TOSTRING(MODULE_NAME), PYINIT_MODULE_NAME);
+void arch_appendinittab()
+{
+    PyImport_AppendInittab(TOSTRING(MODULE_NAME), PYINIT_MODULE_NAME);
 }
 
 static wchar_t *program;
 
-void init_python(const char *executable) {
-	program = Py_DecodeLocale(executable, NULL);
-	if (program == NULL) {
-		fprintf(stderr, "Fatal error: cannot decode executable filename\n");
-		exit(1);
-	}
-	try {
-		PyImport_AppendInittab(TOSTRING(MODULE_NAME), PYINIT_MODULE_NAME);
-		emb::append_inittab();
-		Py_SetProgramName(program);
-		Py_Initialize();
-		PyImport_ImportModule(TOSTRING(MODULE_NAME));
-	} catch (boost::python::error_already_set const &) {
-		// Parse and output the exception
-		std::string perror_str = parse_python_exception();
-		std::cout << "Error in Python: " << perror_str << std::endl;
-	}
+void init_python(const char *executable)
+{
+    program = Py_DecodeLocale(executable, NULL);
+    if (program == NULL) {
+        fprintf(stderr, "Fatal error: cannot decode executable filename\n");
+        exit(1);
+    }
+    try {
+        PyImport_AppendInittab(TOSTRING(MODULE_NAME), PYINIT_MODULE_NAME);
+        emb::append_inittab();
+        Py_SetProgramName(program);
+        Py_Initialize();
+        PyImport_ImportModule(TOSTRING(MODULE_NAME));
+    } catch (boost::python::error_already_set const &) {
+        // Parse and output the exception
+        std::string perror_str = parse_python_exception();
+        std::cout << "Error in Python: " << perror_str << std::endl;
+    }
 }
 
-void deinit_python() {
-	Py_Finalize();
-	PyMem_RawFree(program);
+void deinit_python()
+{
+    Py_Finalize();
+    PyMem_RawFree(program);
 }
 
-void execute_python_file(const char *python_file) {
-	try {
-		FILE *fp = fopen(python_file, "r");
-		if (fp == NULL) {
-			fprintf(stderr, "Fatal error: file not found %s\n", python_file);
-			exit(1);
-		}
-		PyRun_SimpleFile(fp, python_file);
-		fclose(fp);
-	}
-	catch (boost::python::error_already_set const &) {
-		// Parse and output the exception
-		std::string perror_str = parse_python_exception();
-		std::cout << "Error in Python: " << perror_str << std::endl;
-	}
+void execute_python_file(const char *python_file)
+{
+    try {
+        FILE *fp = fopen(python_file, "r");
+        if (fp == NULL) {
+            fprintf(stderr, "Fatal error: file not found %s\n", python_file);
+            exit(1);
+        }
+        PyRun_SimpleFile(fp, python_file);
+        fclose(fp);
+    } catch (boost::python::error_already_set const &) {
+        // Parse and output the exception
+        std::string perror_str = parse_python_exception();
+        std::cout << "Error in Python: " << perror_str << std::endl;
+    }
 }
-
diff --git a/common/pybindings.h b/common/pybindings.h
index a99ad51b..bb060718 100644
--- a/common/pybindings.h
+++ b/common/pybindings.h
@@ -23,85 +23,85 @@
 
 #include "pycontainers.h"
 
-#include <utility>
-#include <stdexcept>
+#include <Python.h>
 #include <boost/python.hpp>
-#include <boost/python/suite/indexing/vector_indexing_suite.hpp>
-#include <boost/python/suite/indexing/map_indexing_suite.hpp>
 #include <boost/python/suite/indexing/map_indexing_suite.hpp>
-#include <Python.h>
+#include <boost/python/suite/indexing/vector_indexing_suite.hpp>
+#include <stdexcept>
+#include <utility>
 using namespace boost::python;
 
 /*
 A wrapper to enable custom type/ID to/from string conversions
  */
-template<typename T>
-struct string_wrapper {
-	template<typename F>
-	struct from_pystring_converter {
-		from_pystring_converter() {
-			converter::registry::push_back(
-					&convertible,
-					&construct,
-					boost::python::type_id<T>());
-		};
-
-		static void *convertible(PyObject *object) {
-			return PyUnicode_Check(object) ? object : 0;
-		}
-
-		static void construct(
-				PyObject *object,
-				converter::rvalue_from_python_stage1_data *data) {
-			const wchar_t *value = PyUnicode_AsUnicode(object);
-			const std::wstring value_ws(value);
-			if (value == 0) throw_error_already_set();
-			void *storage = (
-					(boost::python::converter::rvalue_from_python_storage<T> *)
-							data)->storage.bytes;
-			new(storage) T(fn(std::string(value_ws.begin(), value_ws.end())));
-			data->convertible = storage;
-		}
-
-		static F fn;
-	};
-
-	template<typename F>
-	struct to_str_wrapper {
-		static F fn;
-
-		std::string str(T &x) {
-			return fn(x);
-		}
-	};
-
-	template<typename F1, typename F2>
-	static void wrap(const char *type_name, F1 to_str_fn, F2 from_str_fn) {
-		from_pystring_converter<F2>::fn = from_str_fn;
-		from_pystring_converter<F2>();
-		to_str_wrapper<F1>::fn = to_str_fn;
-		class_<T>(type_name, no_init).def("__str__", to_str_wrapper<F1>::str);
-	};
+template <typename T> struct string_wrapper
+{
+    template <typename F> struct from_pystring_converter
+    {
+        from_pystring_converter()
+        {
+            converter::registry::push_back(&convertible, &construct,
+                                           boost::python::type_id<T>());
+        };
+
+        static void *convertible(PyObject *object)
+        {
+            return PyUnicode_Check(object) ? object : 0;
+        }
+
+        static void construct(PyObject *object,
+                              converter::rvalue_from_python_stage1_data *data)
+        {
+            const wchar_t *value = PyUnicode_AsUnicode(object);
+            const std::wstring value_ws(value);
+            if (value == 0)
+                throw_error_already_set();
+            void *storage =
+                    ((boost::python::converter::rvalue_from_python_storage<T> *)
+                             data)
+                            ->storage.bytes;
+            new (storage) T(fn(std::string(value_ws.begin(), value_ws.end())));
+            data->convertible = storage;
+        }
+
+        static F fn;
+    };
+
+    template <typename F> struct to_str_wrapper
+    {
+        static F fn;
+
+        std::string str(T &x) { return fn(x); }
+    };
+
+    template <typename F1, typename F2>
+    static void wrap(const char *type_name, F1 to_str_fn, F2 from_str_fn)
+    {
+        from_pystring_converter<F2>::fn = from_str_fn;
+        from_pystring_converter<F2>();
+        to_str_wrapper<F1>::fn = to_str_fn;
+        class_<T>(type_name, no_init).def("__str__", to_str_wrapper<F1>::str);
+    };
 };
 
 std::string parse_python_exception();
 
-template<typename Tn>
-void python_export_global(const char *name, Tn &x) {
-	PyObject * m, *d;
-	m = PyImport_AddModule("__main__");
-	if (m == NULL)
-		return;
-	d = PyModule_GetDict(m);
-	try {
-		PyObject * p = incref(object(boost::ref(x)).ptr());
-		PyDict_SetItemString(d, name, p);
-	} catch (boost::python::error_already_set const &) {
-		// Parse and output the exception
-		std::string perror_str = parse_python_exception();
-		std::cout << "Error in Python: " << perror_str << std::endl;
-		std::terminate();
-	}
+template <typename Tn> void python_export_global(const char *name, Tn &x)
+{
+    PyObject *m, *d;
+    m = PyImport_AddModule("__main__");
+    if (m == NULL)
+        return;
+    d = PyModule_GetDict(m);
+    try {
+        PyObject *p = incref(object(boost::ref(x)).ptr());
+        PyDict_SetItemString(d, name, p);
+    } catch (boost::python::error_already_set const &) {
+        // Parse and output the exception
+        std::string perror_str = parse_python_exception();
+        std::cout << "Error in Python: " << perror_str << std::endl;
+        std::terminate();
+    }
 };
 
 void init_python(const char *executable);