Try 1 to fix the mac build

Author: gitbuda

include/mgclient.h

@@ -137,17 +137,17 @@ extern "C" {
 /// Client software version.
 ///
 /// \return Client version in the major.minor.patch format.
-MGCLIENT_EXPORT const char *mg_client_version();
+MGCLIENT_EXPORT const char *mg_client_version(void);
 
 /// Initializes the client (the whole process).
 /// Should be called at the beginning of each process using the client.
 ///
 /// \return Zero if initialization was successful.
-MGCLIENT_EXPORT int mg_init();
+MGCLIENT_EXPORT int mg_init(void);
 
 /// Finalizes the client (the whole process).
 /// Should be called at the end of each process using the client.
-MGCLIENT_EXPORT void mg_finalize();
+MGCLIENT_EXPORT void mg_finalize(void);
 
 /// An enum listing all the types as specified by Bolt protocol.
 enum mg_value_type {
@@ -297,7 +297,7 @@ typedef struct mg_point_3d mg_point_3d;
 /// Constructs a nil \ref mg_value.
 ///
 /// \return Pointer to the newly constructed value or NULL if error occurred.
-MGCLIENT_EXPORT mg_value *mg_value_make_null();
+MGCLIENT_EXPORT mg_value *mg_value_make_null(void);
 
 /// Constructs a boolean \ref mg_value.
 ///
@@ -1221,7 +1221,7 @@ typedef int (*mg_trust_callback_type)(const char *, const char *, const char *,
                                       const char *, void *);
 
 /// Creates a new `mg_session_params` object.
-MGCLIENT_EXPORT mg_session_params *mg_session_params_make();
+MGCLIENT_EXPORT mg_session_params *mg_session_params_make(void);
 
 /// Destroys a `mg_session_params` object.
 MGCLIENT_EXPORT void mg_session_params_destroy(mg_session_params *);

src/apple/mgsocket.c

@@ -16,8 +16,6 @@
 
 #include <string.h>
 
-#include "mgcommon.h"
-
 #define MG_RETRY_ON_EINTR(expression)          \
   __extension__({                              \
     long result;                               \
@@ -27,7 +25,7 @@
     result;                                    \
   })
 
-int mg_socket_init() { return MG_SUCCESS; }
+int mg_socket_init(void) { return MG_SUCCESS; }
 
 int mg_socket_create(int af, int type, int protocol) {
   int sockfd = socket(af, type, protocol);
@@ -92,6 +90,6 @@ int mg_socket_pair(int d, int type, int protocol, int *sv) {
 
 int mg_socket_close(int sock) { return MG_RETRY_ON_EINTR(close(sock)); }
 
-char *mg_socket_error() { return strerror(errno); }
+char *mg_socket_error(void) { return strerror(errno); }
 
-void mg_socket_finalize() {}
+void mg_socket_finalize(void) {}

src/linux/mgsocket.c

@@ -17,8 +17,6 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include "mgcommon.h"
-
 #ifdef __EMSCRIPTEN__
 #include "emscripten.h"
 #include "mgwasm.h"
@@ -36,7 +34,7 @@
 // Please refer to https://man7.org/linux/man-pages/man2 for more details about
 // Linux system calls.
 
-int mg_socket_init() { return MG_SUCCESS; }
+int mg_socket_init(void) { return MG_SUCCESS; }
 
 int mg_socket_create(int af, int type, int protocol) {
   int sockfd = socket(af, type, protocol);
@@ -151,6 +149,6 @@ int mg_socket_pair(int d, int type, int protocol, int *sv) {
 
 int mg_socket_close(int sock) { return MG_RETRY_ON_EINTR(close(sock)); }
 
-char *mg_socket_error() { return strerror(errno); }
+char *mg_socket_error(void) { return strerror(errno); }
 
-void mg_socket_finalize() {}
+void mg_socket_finalize(void) {}

src/mgclient.c

@@ -29,9 +29,9 @@
 #include "mgtransport.h"
 #include "mgvalue.h"
 
-const char *mg_client_version() { return MGCLIENT_VERSION; }
+const char *mg_client_version(void) { return MGCLIENT_VERSION; }
 
-int mg_init_session_static_vars() {
+int mg_init_session_static_vars(void) {
   mg_value *n_val = mg_value_make_integer(-1);
   if (!n_val) {
     goto fatal_failure;
@@ -55,13 +55,13 @@ int mg_init_session_static_vars() {
   return MG_ERROR_CLIENT_ERROR;
 }
 
-int mg_init() {
+int mg_init(void) {
   int init_status = mg_init_session_static_vars();
   if (init_status != 0) return init_status;
   return mg_socket_init();
 }
 
-void mg_finalize() { mg_socket_finalize(); }
+void mg_finalize(void) { mg_socket_finalize(); }
 
 typedef struct mg_session_params {
   const char *address;
@@ -78,7 +78,7 @@ typedef struct mg_session_params {
   void *trust_data;
 } mg_session_params;
 
-mg_session_params *mg_session_params_make() {
+mg_session_params *mg_session_params_make(void) {
   mg_session_params *params =
       mg_allocator_malloc(&mg_system_allocator, sizeof(mg_session_params));
   if (!params) {

src/mgsocket.h

@@ -54,7 +54,7 @@ typedef long ssize_t;
 
 /// Initializes underlying resources. Has to be called at the beginning of a
 /// process using socket resources.
-int mg_socket_init();
+int mg_socket_init(void);
 
 /// Returns a descriptor referencing the new socket or MG_ERROR_SOCKET in the
 /// case of any failure.
@@ -110,11 +110,11 @@ int mg_socket_close(int sock);
 
 /// Used to get a native error message after some socket call fails.
 /// Has to be called immediately after the failed socket function.
-char *mg_socket_error();
+char *mg_socket_error(void);
 
 /// Should be called at the end of any process which previously called the
 /// \ref mg_socket_init function.
-void mg_socket_finalize();
+void mg_socket_finalize(void);
 
 #ifdef __cplusplus
 }

src/mgtransport.c

@@ -15,7 +15,6 @@
 #include "mgtransport.h"
 
 #include <assert.h>
-#include <errno.h>
 #include <stdlib.h>
 #include <string.h>
 #ifdef MGCLIENT_ON_LINUX
@@ -26,7 +25,6 @@
 
 #include "mgallocator.h"
 #include "mgclient.h"
-#include "mgcommon.h"
 #include "mgsocket.h"
 #ifdef __EMSCRIPTEN__
 #include "mgwasm.h"
@@ -160,7 +158,7 @@ static char *hex_encode(unsigned char *data, unsigned int len,
   return encoded;
 }
 
-static void mg_openssl_init() {
+static void mg_openssl_init(void) {
 #if OPENSSL_VERSION_NUMBER < 0x10100000L
   static pthread_mutex_t mutex = PTHREAD_MUTEX_INITIALIZER;
   static int mg_ssl_initialized = 0;

src/mgvalue.c

@@ -19,7 +19,6 @@
 
 #include "mgallocator.h"
 #include "mgclient.h"
-#include "mgconstants.h"
 
 mg_string *mg_string_alloc(uint32_t size, mg_allocator *allocator) {
   char *block = mg_allocator_malloc(allocator, sizeof(mg_string) + size);
@@ -178,7 +177,7 @@ mg_point_3d *mg_point_3d_alloc(mg_allocator *allocator) {
   return point_3d;
 }
 
-mg_value *mg_value_make_null() {
+mg_value *mg_value_make_null(void) {
   mg_value *value = mg_allocator_malloc(&mg_system_allocator, sizeof(mg_value));
   if (!value) {
     return NULL;