Change LOG() macro to be varadic

This changes the LOG macro to be varadic removing the need for all
callsites to have double bracketing and allows for future improvement
on how we use the logging macros.

The callsites were changed with coccinelle and the changes checked by
hand. Compile tested for several frontends but not all.

A formatting annotation has also been added which allows the compiler
to check the parameters and types passed to the logging.

1 files changed, 6 insertions, 6 deletions

diff --git a/atari/search.c b/atari/search.c
index c2b02f06a..8ba552198 100644
--- a/atari/search.c
+++ b/atari/search.c
@@ -67,7 +67,7 @@ struct gui_search_table *atari_search_table = &search_table;
  */
 void nsatari_search_set_status(bool found, void *p)
 {
-	LOG(("%p set status: %d\n", p, found));
+	LOG("%p set status: %d\n", p, found);
 	// TODO: maybe update GUI
 }
 
@@ -80,7 +80,7 @@ void nsatari_search_set_status(bool found, void *p)
 void nsatari_search_set_hourglass(bool active, void *p)
 {
 	SEARCH_FORM_SESSION s = (SEARCH_FORM_SESSION)p;
-	LOG(("active: %d, session: %p", active, p));
+	LOG("active: %d, session: %p", active, p);
 	if (active) {
 		gui_window_set_pointer(s->g, GUI_POINTER_PROGRESS);
 	} else {
@@ -99,7 +99,7 @@ void nsatari_search_set_hourglass(bool active, void *p)
  */
 void nsatari_search_add_recent(const char *string, void *p)
 {
-	LOG(("%p add recent: %s\n", p, string));
+	LOG("%p add recent: %s\n", p, string);
 }
 
 
@@ -115,7 +115,7 @@ void nsatari_search_set_forward_state(bool active, void *p)
 	GRECT area;
 	SEARCH_FORM_SESSION s = (SEARCH_FORM_SESSION)p;
 	/* deactivate back cb */
-	LOG(("%p: set forward state: %d\n", p, active));
+	LOG("%p: set forward state: %d\n", p, active);
 
 	gw = s->g;
 
@@ -142,7 +142,7 @@ void nsatari_search_set_back_state(bool active, void *p)
 	GRECT area;
 	SEARCH_FORM_SESSION s = (SEARCH_FORM_SESSION)p;
 	/* deactivate back cb */
-	LOG(("%p: set back state: %d\n", p, active));
+	LOG("%p: set back state: %d\n", p, active);
 
 	s->state.back_avail = active;
 	gw = s->g;
@@ -224,7 +224,7 @@ void nsatari_search_restore_form( struct s_search_form_session *s, OBJECT *obj)
 void nsatari_search_session_destroy(struct s_search_form_session *s)
 {
 	if (s != NULL) {
-		LOG((""));
+		LOG("");
 		browser_window_search_clear(s->g->browser->bw);
 		free(s);
 	}