remove submodule, add source files (#134)

* remove submodule

* add alex-c-collections, remove submodule

Author: alex-courtis

.cppcheck.supp

@@ -1,2 +1,3 @@
+unusedFunction:lib/col/*c
 unusedFunction:tst/wrap-log.c
 unusedFunction:tst/util.c

.github/workflows/ci.yml

@@ -11,8 +11,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v3
-      with:
-        submodules: true
     - name: deps
       run: |
         sudo add-apt-repository universe

.gitmodules

@@ -20,12 +20,6 @@ Most will be available if you are running a wlroots based compositor like sway.
 
 yaml-cpp will need to be installed via your distribution's package manager.
 
-## Library Submodules
-
-Following clone the libraries must be fetched.
-
-`git submodule update --init`
-
 ## Development
 
 gcc is the default for packaging reasons, however clang is preferred.

CONTRIBUTING.md

@@ -1,15 +1,11 @@
 include config.mk
 
-INC_H = $(wildcard inc/*.h)
+INC_H = $(wildcard inc/*.h) $(wildcard lib/col/inc/*.h)
 
-SRC_C = $(wildcard src/*.c)
+SRC_C = $(wildcard src/*.c) $(wildcard lib/col/src/*.c)
 SRC_CXX = $(wildcard src/*.cpp)
 SRC_O = $(SRC_C:.c=.o) $(SRC_CXX:.cpp=.o)
 
-LIB_H = $(wildcard lib/alex-c-collections/inc/*.h)
-LIB_C = $(wildcard lib/alex-c-collections/src/*.c)
-LIB_O = $(LIB_C:.c=.o)
-
 EXAMPLE_C = $(wildcard examples/*.c)
 EXAMPLE_O = $(EXAMPLE_C:.c=.o)
 EXAMPLE_E = $(EXAMPLE_C:.c=)
@@ -28,11 +24,10 @@ TST_T = $(patsubst tst%,test%,$(TST_E))
 all: way-displays
 
 $(SRC_O): $(INC_H) $(PRO_H) config.mk GNUmakefile
-$(LIB_O): $(LIB_H) $(LIB_C) config.mk GNUmakefile
 $(PRO_O): $(PRO_H) config.mk GNUmakefile
 $(EXAMPLE_O): $(INC_H) $(PRO_H) config.mk GNUmakefile
 
-way-displays: $(SRC_O) $(PRO_O) $(LIB_O)
+way-displays: $(SRC_O) $(PRO_O)
 	$(CXX) -o $(@) $(^) $(LDFLAGS) $(LDLIBS)
 
 $(PRO_H): $(PRO_X)
@@ -42,7 +37,7 @@ $(PRO_C): $(PRO_X)
 	wayland-scanner private-code $(@:.c=.xml) $@
 
 clean:
-	rm -f way-displays $(SRC_O) $(PRO_O) $(PRO_H) $(PRO_C) $(LIB_O) $(TST_O) $(TST_E) $(EXAMPLE_E) $(EXAMPLE_O) 
+	rm -f way-displays $(SRC_O) $(PRO_O) $(PRO_H) $(PRO_C) $(TST_O) $(TST_E) $(EXAMPLE_E) $(EXAMPLE_O)
 
 install: way-displays way-displays.1 cfg.yaml
 	mkdir -p $(DESTDIR)$(PREFIX)/bin
@@ -84,7 +79,7 @@ $(TST_T): all
 	$(VALGRIND) ./$(EXE)
 
 examples: $(EXAMPLE_E)
-examples/%: examples/%.o $(filter-out src/main.o,$(SRC_O)) $(PRO_O) $(LIB_O)
+examples/%: examples/%.o $(filter-out src/main.o,$(SRC_O)) $(PRO_O)
 	$(CXX) -o $(@) $(^) $(LDFLAGS) $(LDLIBS)
 
 .PHONY: all clean install uninstall man cppcheck iwyu test test-vg $(TST_T)

GNUmakefile

@@ -4,7 +4,7 @@ PREFIX ?= /usr/local
 PREFIX_ETC ?= /usr/local
 ROOT_ETC ?= /etc
 
-INCS = -Iinc -Ipro -Ilib/alex-c-collections/inc
+INCS = -Iinc -Ipro -Ilib/col/inc
 
 CPPFLAGS += $(INCS) -D_GNU_SOURCE -DVERSION=\"$(VERSION)\" -DROOT_ETC=\"$(ROOT_ETC)\"
 

config.mk

@@ -0,0 +1 @@
+[email protected]:alex-courtis/alex-c-collections.git v1.1.1

lib/alex-c-collections

@@ -0,0 +1,67 @@
+#ifndef ITABLE_H
+#define ITABLE_H
+
+#include <stddef.h>
+#include <stdint.h>
+
+/*
+ * Array backed integer indexed table.
+ * Entries preserve insertion order.
+ * Operations linearly traverse keys.
+ * NULL values permitted.
+ * Not thread safe.
+ */
+struct ITable;
+
+/*
+ * Entry iterator.
+ */
+struct ITableIter {
+	const uint64_t key;
+	const void* const val;
+};
+
+/*
+ * Lifecycle
+ */
+
+// construct a table with initial size, growing as necessary, NULL on zero param
+const struct ITable *itable_init(const size_t initial, const size_t grow);
+
+// free table
+void itable_free(const void* const tab);
+
+// free table and vals, null free_val uses free()
+void itable_free_vals(const struct ITable* const tab, void (*free_val)(const void* const val));
+
+// free iter
+void itable_iter_free(const struct ITableIter* const iter);
+
+/*
+ * Access
+ */
+
+// return val, NULL not present
+const void *itable_get(const struct ITable* const tab, const uint64_t key);
+
+// create an iterator, caller must itable_iter_free or invoke itable_next until NULL
+const struct ITableIter *itable_iter(const struct ITable* const tab);
+
+// next iterator value, NULL at end of list
+const struct ITableIter *itable_next(const struct ITableIter* const iter);
+
+// number of entries with val
+size_t itable_size(const struct ITable* const tab);
+
+/*
+ * Mutate
+ */
+
+// set key/val, return old val if overwritten
+const void *itable_put(const struct ITable* const tab, const uint64_t key, const void* const val);
+
+// remove key, return old val if present
+const void *itable_remove(const struct ITable* const tab, const uint64_t key);
+
+#endif // ITABLE_H
+

lib/col/.source

@@ -0,0 +1,65 @@
+#ifndef OSET_H
+#define OSET_H
+
+#include <stdbool.h>
+#include <stddef.h>
+
+/*
+ * Array backed ordered set.
+ * Operations linearly traverse values.
+ * NULL not permitted.
+ * Not thread safe.
+*/
+struct OSet;
+
+/*
+ * Entry iterator.
+ */
+struct OSetIter {
+	const void* const val;
+};
+
+/*
+ * Lifecycle
+ */
+
+// construct a set with initial size, grow as needed, NULL on zero param
+const struct OSet *oset_init(const size_t initial, const size_t grow);
+
+// free set
+void oset_free(const void* const set);
+
+// free map and vals, NULL free_val uses free()
+void oset_free_vals(const struct OSet* const set, void (*free_val)(const void* const val));
+
+// free iter
+void oset_iter_free(const struct OSetIter* const iter);
+
+/*
+ * Access
+ */
+
+// true if this set contains the specified element
+bool oset_contains(const struct OSet* const set, const void* const val);
+
+// number of values
+size_t oset_size(const struct OSet* const set);
+
+// create an iterator, caller must oset_iter_free or invoke oset_next until NULL
+const struct OSetIter *oset_iter(const struct OSet* const set);
+
+// next iterator value, NULL at end of set
+const struct OSetIter *oset_next(const struct OSetIter* const iter);
+
+/*
+ * Mutate
+ */
+
+// true if this set did not already contain the specified element
+bool oset_add(const struct OSet* const set, const void* const val);
+
+// true if this set contained the element
+bool oset_remove(const struct OSet* const set, const void* const val);
+
+#endif // OSET_H
+