From cd917f48454875ad6b7fc69455281d72760c44ee Mon Sep 17 00:00:00 2001
From: Egor Tensin <Egor.Tensin@gmail.com>
Date: Sat, 13 May 2023 10:58:41 +0200
Subject: best practices & coding style fixes

* I don't really need to declare all variables at the top of the
  function anymore.
* Default-initialize variables more.
* Don't set the output parameter until the object is completely
  constructed.
---
 src/client_main.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

(limited to 'src/client_main.c')

diff --git a/src/client_main.c b/src/client_main.c
index dd22508..c402b20 100644
--- a/src/client_main.c
+++ b/src/client_main.c
@@ -10,6 +10,7 @@
 #include "const.h"
 
 #include <getopt.h>
+#include <stdlib.h>
 
 static struct settings default_settings()
 {
@@ -62,7 +63,7 @@ static int parse_settings(struct settings *settings, int argc, char *argv[])
 int main(int argc, char *argv[])
 {
 	struct settings settings;
-	struct client *client;
+	struct client *client = NULL;
 	int ret = 0;
 
 	ret = parse_settings(&settings, argc, argv);
-- 
cgit v1.2.3