[#422] Add short comments, logic fix

src/libpgagroal/configuration.c

@@ -2397,15 +2397,19 @@ is_same_server(struct server* s1, struct server* s2)
    }
 }
 
+/**
+ * Checks if TLS configurations are same.
+ * @return true if the TLS configurations are same
+*/
 static bool
 is_same_tls(struct server* src, struct server* dst)
 {
-   if (src->tls == dst->tls ||
-       !strncmp(src->tls_cert_file, dst->tls_cert_file, MISC_LENGTH)||
-       !strncmp(src->tls_key_file, dst->tls_key_file, MISC_LENGTH)||
+   if (src->tls == dst->tls &&
+       !strncmp(src->tls_cert_file, dst->tls_cert_file, MISC_LENGTH) &&
+       !strncmp(src->tls_key_file, dst->tls_key_file, MISC_LENGTH) &&
        !strncmp(src->tls_ca_file, dst->tls_ca_file, MISC_LENGTH))
    {
-      return false;
+      return true;
    }
    else
    {
@@ -2464,6 +2468,10 @@ restart_int(char* name, int e, int n)
    return 0;
 }
 
+/**
+ * Utility function prints a line in the log when a restart is required.
+ * @return 0 when parameter values are same, 1 when a restart required.
+*/
 static int
 restart_bool(char* name, bool e, bool n)
 {