Merge pull request #27 from urielgarciarivas/main

Enhance C and C++ flags in Makefiles
zoningorg · May 10, 2024 · 05af01e · 05af01e
2 parents 9049c85 + c999494
commit 05af01e
Show file tree

Hide file tree

Showing 7 changed files with 17 additions and 16 deletions.
diff --git a/DataStructures/makefile b/DataStructures/makefile
@@ -15,7 +15,7 @@
 # https://github.com/zoningorg/zoning/blob/main/LICENSE
 
 CC = gcc
-CFLAGS = -Wall -Werror -Wpedantic -I./inc/
+CFLAGS = -Wall -Wextra -Werror -Wpedantic -I./inc/
 INC = ./inc/array.h ./inc/hash_map.h ./inc/hash_set.h ./inc/linked_list.h ./inc/sort_map.h ./inc/sort_set.h
 SRC = $(patsubst ./inc/%.h,./src/%.c,$(INC))
 OBJ = $(patsubst ./inc/%.h,./obj/%.o,$(INC))

diff --git a/DataStructures/src/linked_list.c b/DataStructures/src/linked_list.c
@@ -139,7 +139,7 @@ void erase_single_match_linked_list(linked_list* list, int target) {
   }
 }
 
-void erase_all_match_linked_list(linked_list* list, int target) {}
+//void erase_all_match_linked_list(linked_list* list, int target) {}
 
 void reverse_linked_list(linked_list*const list) {
   if (list == NULL) {
@@ -201,7 +201,7 @@ bool are_equal_linked_list(
   return lhs_node == NULL && rhs_node == NULL;
 }
 
-void sort_linked_list(linked_list*const list) {}
+//void sort_linked_list(linked_list*const list) {}
 
 inline bool is_empty_linked_list(const linked_list*const list) {
   return list == NULL

diff --git a/DataStructures/src/sort_set.c b/DataStructures/src/sort_set.c
@@ -34,7 +34,7 @@ inline sort_set* new_sort_set(int value) {
   return response;
 }
 
-inline void add_to_sort_set(sort_set*const set, int value) {}
+//inline void add_to_sort_set(sort_set*const set, int value) { set; }
 
 bool exist_in_sort_set(const sort_set*const set, int target) {
   if (set == NULL || set->size == 0) {

diff --git a/DataStructures/test/sort_set_test.c b/DataStructures/test/sort_set_test.c
@@ -30,7 +30,7 @@ int main(void) {
   sort_set* set = new_sort_set(1);
 
   for (int i = first_element + 1; i <= last_element; ++i) {
-    add_to_sort_set(set, i);
+    //add_to_sort_set(set, i);
   }
 
   delete_sort_set(set);

diff --git a/FileTransfer/PaqueteDatagrama.h b/FileTransfer/PaqueteDatagrama.h
@@ -60,8 +60,8 @@ class PaqueteDatagrama final {
   size_t longitud() const noexcept { return longitud_; }
 
  private:
-  std::string ip_;
-  unsigned short puerto_;
   Mensaje mensaje_;
   size_t longitud_;
+  std::string ip_;
+  unsigned short puerto_;
 };
diff --git a/FileTransfer/makefile b/FileTransfer/makefile
@@ -14,28 +14,31 @@
 #
 # https://github.com/zoningorg/zoning/blob/main/LICENSE
 
+CXX = g++
+CXXFLAGS = -Wall -Wextra -Werror -Wpedantic
+
 all: ENVIAR_ARCHIVOS RECIBIR_ARCHIVOS
 
 ENVIAR_ARCHIVOS: Cliente.cpp Solicitud.o SocketDatagrama.o
-	g++ Cliente.cpp Solicitud.o SocketDatagrama.o -o ENVIAR_ARCHIVOS
+	$(CXX) $(CXXFLAGS) Cliente.cpp Solicitud.o SocketDatagrama.o -o ENVIAR_ARCHIVOS
 
 RECIBIR_ARCHIVOS: Servidor.cpp Respuesta.o SocketDatagrama.o
-	g++ Servidor.cpp Respuesta.o SocketDatagrama.o -o RECIBIR_ARCHIVOS
+	$(CXX) $(CXXFLAGS) Servidor.cpp Respuesta.o SocketDatagrama.o -o RECIBIR_ARCHIVOS
 
 Respuesta.o: Respuesta.cpp Mensaje.h
-	g++ Respuesta.cpp -c
+	$(CXX) $(CXXFLAGS) Respuesta.cpp -c
 
 Solicitud.o: Solicitud.cpp Mensaje.h
-	g++ Solicitud.cpp -c
+	$(CXX) $(CXXFLAGS) Solicitud.cpp -c
 
 SocketDatagrama.o: SocketDatagrama.cpp
-	g++ SocketDatagrama.cpp -c
+	$(CXX) $(CXXFLAGS) SocketDatagrama.cpp -c
 
 test: PaqueteDatagrama_test
 	./PaqueteDatagrama_test ; rm PaqueteDatagrama_test
 
 PaqueteDatagrama_test: test/PaqueteDatagrama_test.cpp
-	g++ test/PaqueteDatagrama_test.cpp -o PaqueteDatagrama_test
+	$(CXX) $(CXXFLAGS) test/PaqueteDatagrama_test.cpp -o PaqueteDatagrama_test
 
 clean:
 	rm -f $(wildcard *.o) ENVIAR_ARCHIVOS RECIBIR_ARCHIVOS
diff --git a/FileTransfer/test/PaqueteDatagrama_test.cpp b/FileTransfer/test/PaqueteDatagrama_test.cpp
@@ -16,12 +16,10 @@
  * https://github.com/zoningorg/zoning/blob/main/LICENSE
  */
 
-#include <iostream>
-
 #include "../../Testing/comparators.h"
 #include "../PaqueteDatagrama.h"
 
-int main(int argc, char** argv) {
+int main(void) {
   START_TEST("PaqueteDatagrama_test");
 
   Mensaje mensaje;