From 083b2473292e54ba90355d0dc350ef3f2f7fda6e Mon Sep 17 00:00:00 2001
From: Timerix <timerix11@gmail.com>
Date: Sun, 9 Nov 2025 18:39:50 +0500
Subject: [PATCH] enabled more warnings

---
 dependencies/tlibc                       |  2 +-
 project.config                           | 13 +++++++++++--
 src/client/ClientCredentials.c           |  4 ++--
 src/client/ServerConnection.c            |  2 +-
 src/client/client.c                      |  4 +++-
 src/client/client.h                      |  3 +--
 src/server/request_handlers/send_error.c |  2 ++
 7 files changed, 21 insertions(+), 9 deletions(-)

diff --git a/dependencies/tlibc b/dependencies/tlibc
index 2c8e6fc..4cc226b 160000
--- a/dependencies/tlibc
+++ b/dependencies/tlibc
@@ -1 +1 @@
-Subproject commit 2c8e6fc601a868851d8ce50f77b391e6a9b7e656
+Subproject commit 4cc226b57af3691b9c6062499ce0a18755047c59
diff --git a/project.config b/project.config
index 99d1104..cc0a9ff 100644
--- a/project.config
+++ b/project.config
@@ -6,8 +6,17 @@ CMP_C="gcc"
 CMP_CPP="g++"
 STD_C="c99"
 STD_CPP="c++11"
-WARN_C="-Wall -Wextra -Werror=return-type -Werror=pointer-arith -Wno-unused-parameter"
-WARN_CPP="-Wall -Wextra -Werror=return-type -Werror=pointer-arith -Wno-unused-parameter"
+WARN_C="-Wall -Wextra
+    -Wduplicated-branches
+    -Wduplicated-cond
+    -Wformat=2
+    -Wmissing-include-dirs
+    -Wshadow
+    -Werror=return-type
+    -Werror=pointer-arith
+    -Werror=init-self
+    -Werror=incompatible-pointer-types"
+WARN_CPP="$WARN_C"
 SRC_C="$(find src -name '*.c')"
 #SRC_CPP="$(find src -name '*.cpp')"
 
diff --git a/src/client/ClientCredentials.c b/src/client/ClientCredentials.c
index 324d8cb..4f7c5fa 100644
--- a/src/client/ClientCredentials.c
+++ b/src/client/ClientCredentials.c
@@ -29,12 +29,12 @@ Result(void) ClientCredentials_tryConstruct(ClientCredentials* cred,
     
     // lvl 1 hash - is used as AES key for user data
     cred->user_data_key = Array_alloc(u8, PASSWORD_HASH_SIZE);
-    hash_password(List_castTo_Array(data_to_hash), cred->user_data_key.data, __PASSWORD_HASH_LVL_ROUNDS);
+    hash_password(List_castTo_Array(data_to_hash), cred->user_data_key.data, PASSWORD_HASH_LVL_ROUNDS);
     // concat lvl 1 hash to data_to_hash
     List_push_size(&data_to_hash, cred->user_data_key.data, cred->user_data_key.size);
     // lvl 2 hash - is used for authentification
     cred->token = Array_alloc(u8, PASSWORD_HASH_SIZE);
-    hash_password(List_castTo_Array(data_to_hash), cred->token.data, __PASSWORD_HASH_LVL_ROUNDS);
+    hash_password(List_castTo_Array(data_to_hash), cred->token.data, PASSWORD_HASH_LVL_ROUNDS);
 
     AESBlockEncryptor_construct(&cred->user_data_aes_enc, cred->user_data_key, AESBlockEncryptor_DEFAULT_CLASS);
     AESBlockDecryptor_construct(&cred->user_data_aes_dec, cred->user_data_key, AESBlockDecryptor_DEFAULT_CLASS);
diff --git a/src/client/ServerConnection.c b/src/client/ServerConnection.c
index 4f6ee88..670cda3 100644
--- a/src/client/ServerConnection.c
+++ b/src/client/ServerConnection.c
@@ -41,7 +41,7 @@ Result(void) ServerLink_parse(cstr server_link_cstr, EndpointIPv4* server_end_ou
     Return RESULT_VOID;
 }
 
-Result(ServerConnection*) ServerConnection_open(ClientCredentials* client_credentials, cstr server_link_cstr){
+Result(ServerConnection*) ServerConnection_open(cstr server_link_cstr){
     Deferral(16);
 
     ServerConnection* conn = (ServerConnection*)malloc(sizeof(ServerConnection));
diff --git a/src/client/client.c b/src/client/client.c
index 7b4758e..6326ba9 100644
--- a/src/client/client.c
+++ b/src/client/client.c
@@ -25,6 +25,8 @@ Result(void) Client_createFromConfig(cstr config_path){
     bool success = false;
     Defer(if(!success) Client_free(client));
 
+    (void)config_path;
+
     success = true;
     Return RESULT_VALUE(p, client);
 }
@@ -152,7 +154,7 @@ static Result(void) commandExec(Client* client, str command, bool* stop){
 
         printf("connecting to server...\n");
         try(client->server_connection, p, 
-            ServerConnection_open(&client->cred, new_server_link.data));
+            ServerConnection_open(new_server_link.data));
         printf("connection established\n");
 
         // TODO: request server info
diff --git a/src/client/client.h b/src/client/client.h
index 801994b..1fb47dd 100644
--- a/src/client/client.h
+++ b/src/client/client.h
@@ -28,8 +28,7 @@ typedef struct ServerConnection {
     EncryptedSocketTCP sock;
 } ServerConnection;
 
-Result(ServerConnection*) ServerConnection_open(ClientCredentials* client_credentials,
-    cstr server_link_cstr);
+Result(ServerConnection*) ServerConnection_open(cstr server_link_cstr);
 
 void ServerConnection_close(ServerConnection* conn);
 
diff --git a/src/server/request_handlers/send_error.c b/src/server/request_handlers/send_error.c
index 9c73583..a6eb197 100644
--- a/src/server/request_handlers/send_error.c
+++ b/src/server/request_handlers/send_error.c
@@ -4,6 +4,8 @@ Result(char*) __sendErrorMessage(ClientConnection* conn, PacketHeader* req_head,
     u32 msg_buf_size, cstr format, va_list argv)
 {
     Deferral(4);
+    (void)req_head;
+    
     //TODO: limit ErrorMessage size to fit into EncryptedSocketTCP.internal_buffer_size
     Array(u8) err_buf = Array_alloc(u8, msg_buf_size);
     bool err_complete = false;