Switched C standard from gnu89 to C11 for improved compatibility, modern features, and better code maintainability.

Author: nkallima

CHANGELOG.md

@@ -3,6 +3,7 @@ Changelog
 
 v3.4.0
 ------
+- Switched C standard from `gnu89` to `C11` for improved compatibility, modern features, and better code maintainability.
 - Setting `-flto=auto` in makefiles for better compiling performance
 - Modified the `check-build` and `validate` workflow triggers from 'main' branch only to all branches except `gh-pages` branch.
 

Makefile

@@ -14,16 +14,16 @@ codecov:
 	make $(ARCH) D_ARGS="-DDEBUG -ftest-coverage -fprofile-arcs --coverage -O0"
 
 x86_64:
-	make -f Makefile.generic -j $(NPROC) COMPILER=$(CC) LDLIBS='$(LIBS)' CFLAGS='-Wall -std=gnu89 -Ofast -fPIC -flto=auto -ftree-vectorize -finline-functions -march=native -mtune=native'
+	make -f Makefile.generic -j $(NPROC) COMPILER=$(CC) LDLIBS='$(LIBS)' CFLAGS='-Wall -std=c11 -Ofast -fPIC -flto=auto -ftree-vectorize -finline-functions -march=native -mtune=native -D_XOPEN_SOURCE=700'
 
 icx:
-	make -f Makefile.generic -j $(NPROC) COMPILER=icx LDLIBS='$(LIBS)' CFLAGS='-Wall -std=gnu89 -O3 -fPIC -ftree-vectorize -finline-functions -march=native -mtune=native'
+	make -f Makefile.generic -j $(NPROC) COMPILER=icx LDLIBS='$(LIBS)' CFLAGS='-Wall -std=c11 -O3 -fPIC -ftree-vectorize -finline-functions -march=native -mtune=native -D_XOPEN_SOURCE=700'
 
 clang:
-	make -f Makefile.generic -j $(NPROC) COMPILER=clang AR=$(CLANG_AR) LDLIBS='$(LIBS)' CFLAGS='-Wall -std=gnu89 -Ofast -fPIC -ftree-vectorize -finline-functions -march=native -mtune=native'
+	make -f Makefile.generic -j $(NPROC) COMPILER=clang AR=$(CLANG_AR) LDLIBS='$(LIBS)' CFLAGS='-Wall -std=c11 -Ofast -fPIC -ftree-vectorize -finline-functions -march=native -mtune=native -D_XOPEN_SOURCE=700'
 
 unknown:
-	make -f Makefile.generic -j $(NPROC) COMPILER=$(CC) LDLIBS='$(LIBS)'  CFLAGS='-std=gnu89 -Ofast -fPIC -flto=auto'
+	make -f Makefile.generic -j $(NPROC) COMPILER=$(CC) LDLIBS='$(LIBS)'  CFLAGS='-std=c11 -Ofast -fPIC -flto=auto -D_XOPEN_SOURCE=700'
 
 clean:
 	make -f Makefile.generic clean

libconcurrent/includes/barrier.h

@@ -29,20 +29,20 @@ typedef struct SynchBarrier {
 ///
 /// @param bar A pointer to an instance of the barrier object.
 /// @param n The number of threads that will use the barrier object.
-inline void synchBarrierSet(SynchBarrier *bar, uint32_t n);
+void synchBarrierSet(SynchBarrier *bar, uint32_t n);
 
 /// @brief Whenever a thread executes this function, it waits without returning 
 /// until all the other n-1 threads also execute the same function.
 /// @param bar A pointer to an instance of the barrier object.
-inline void synchBarrierWait(SynchBarrier *bar);
+void synchBarrierWait(SynchBarrier *bar);
 
 /// @brief Whenever a thread executes this function, it states that it will no use this barrier instance anymore. 
 /// Thus, the system may free some resources.
 /// @param bar A pointer to an instance of the barrier object.
-inline void synchBarrierLeave(SynchBarrier *bar);
+void synchBarrierLeave(SynchBarrier *bar);
 
 /// @brief Deprecated functionality. It is only used in the threadtools.c file.
 /// @param bar A pointer to an instance of the barrier object.
-inline void synchBarrierLastLeave(SynchBarrier *bar);
+void synchBarrierLastLeave(SynchBarrier *bar);
 
 #endif

libconcurrent/includes/clhhash.h

@@ -74,15 +74,15 @@ typedef struct CLHHashThreadState {
 /// @param hash A pointer to the hash-table instance.
 /// @param num_cells The number of cells that the hash-table object is going to use.
 /// @param nthreads The number of threads that will use the CLH-Hash object.
-inline void CLHHashStructInit(CLHHash *hash, int num_cells, int nthreads);
+void CLHHashStructInit(CLHHash *hash, int num_cells, int nthreads);
 
 /// @brief This function should be called once before the thread applies any operation to the CLH-Hash combining object.
 ///
 /// @param hash A pointer to the hash-table instance.
 /// @param th_state A pointer to thread's local state of CLH-Hash.
 /// @param num_cells The number of cells that the hash-table object is going to use.
 /// @param pid The pid of the calling thread.
-inline void CLHHashThreadStateInit(CLHHash *hash, CLHHashThreadState *th_state, int num_cells, int pid);
+void CLHHashThreadStateInit(CLHHash *hash, CLHHashThreadState *th_state, int num_cells, int pid);
 
 /// @brief This function tries to insert a <key,value> into the hash-table if there is enough space in the
 /// corresponding cell. If the key already exists in the hash-table, CLHHashInsert updates the corresponding value.
@@ -93,7 +93,7 @@ inline void CLHHashThreadStateInit(CLHHash *hash, CLHHashThreadState *th_state,
 /// @param value The value of the <key,value> pair that CLHHashInsert will try to insert into the hash-table.
 /// @param pid The pid of the calling thread.
 /// @return In case of success true is returned. In case that there is not enough space in the corresponding cell.
-inline bool CLHHashInsert(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int64_t value, int pid);
+bool CLHHashInsert(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int64_t value, int pid);
 
 /// @brief This function searches for a specific key in the hash-table. In case that CLHHashSearch finds the key,
 /// it returns true. Otherwise, it returns false.
@@ -103,7 +103,7 @@ inline bool CLHHashInsert(CLHHash *hash, CLHHashThreadState *th_state, int64_t k
 /// @param key The key of the <key,value> pair that CLHHashSearch will search for.
 /// @param pid The pid of the calling thread.
 /// @return CLHHashSearch returns the value of the <key,value> pair in case that the key exists in the hash-table.
-inline RetVal CLHHashSearch(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int pid);
+RetVal CLHHashSearch(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int pid);
 
 /// @brief This function searches for a specific key in the hash-table. In case that CLHHashDelete finds the key,
 /// it deletes the corresponding <key,value>.
@@ -112,6 +112,6 @@ inline RetVal CLHHashSearch(CLHHash *hash, CLHHashThreadState *th_state, int64_t
 /// @param th_state A pointer to thread's local state of CLH-Hash.
 /// @param key The key of the <key,value> pair that CLHHashDelete will try to delete it.
 /// @param pid The pid of the calling thread.
-inline void CLHHashDelete(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int pid);
+void CLHHashDelete(CLHHash *hash, CLHHashThreadState *th_state, int64_t key, int pid);
 
 #endif

libconcurrent/includes/dsmhash.h

@@ -74,15 +74,15 @@ typedef struct DSMHashThreadState {
 /// @param hash A pointer to the hash-table instance.
 /// @param num_cells The number of cells that the hash-table object is going to use.
 /// @param nthreads The number of threads that will use the DSM-Hash object.
-inline void DSMHashInit(DSMHash *hash, int num_cells, int nthreads);
+void DSMHashInit(DSMHash *hash, int num_cells, int nthreads);
 
 /// @brief This function should be called once before the thread applies any operation to the DSM-Hash combining object.
 ///
 /// @param hash A pointer to the hash-table instance.
 /// @param th_state A pointer to thread's local state of DSM-Hash.
 /// @param num_cells The number of cells that the hash-table object is going to use.
 /// @param pid The pid of the calling thread.
-inline void DSMHashThreadStateInit(DSMHash *hash, DSMHashThreadState *th_state, int num_cells, int pid);
+void DSMHashThreadStateInit(DSMHash *hash, DSMHashThreadState *th_state, int num_cells, int pid);
 
 /// @brief This function tries to insert a <key,value> into the hash-table if there is enough space in the
 /// corresponding cell. If the key already exists in the hash-table, DSMHashInsert updates the corresponding value.
@@ -93,7 +93,7 @@ inline void DSMHashThreadStateInit(DSMHash *hash, DSMHashThreadState *th_state,
 /// @param value The value of the <key,value> pair that DSMHashInsert will try to insert into the hash-table.
 /// @param pid The pid of the calling thread.
 /// @return In case of success true is returned. In case that there is not enough space in the corresponding cell.
-inline bool DSMHashInsert(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int64_t value, int pid);
+bool DSMHashInsert(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int64_t value, int pid);
 
 /// @brief This function searches for a specific key in the hash-table. In case that DSMHashSearch finds the key,
 /// it returns true. Otherwise, it returns false.
@@ -103,7 +103,7 @@ inline bool DSMHashInsert(DSMHash *hash, DSMHashThreadState *th_state, int64_t k
 /// @param key The key of the <key,value> pair that DSMHashSearch will search for.
 /// @param pid The pid of the calling thread.
 /// @return DSMHashSearch returns the value of the <key,value> pair in case that the key exists in the hash-table.
-inline RetVal DSMHashSearch(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int pid);
+RetVal DSMHashSearch(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int pid);
 
 /// @brief This function searches for a specific key in the hash-table. In case that DSMHashDelete finds the key,
 /// it deletes the corresponding <key,value>.
@@ -112,6 +112,6 @@ inline RetVal DSMHashSearch(DSMHash *hash, DSMHashThreadState *th_state, int64_t
 /// @param th_state A pointer to thread's local state of DSM-Hash.
 /// @param key The key of the <key,value> pair that DSMHashDelete will try to delete it.
 /// @param pid The pid of the calling thread.
-inline void DSMHashDelete(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int pid);
+void DSMHashDelete(DSMHash *hash, DSMHashThreadState *th_state, int64_t key, int pid);
 
 #endif