]> git.sur5r.net Git - i3/i3/commitdiff
Use (void) instead of () for functions without args (Thanks fernandotcl)
authorMichael Stapelberg <michael@stapelberg.de>
Sat, 31 Mar 2012 08:53:04 +0000 (10:53 +0200)
committerMichael Stapelberg <michael@stapelberg.de>
Sat, 31 Mar 2012 08:53:04 +0000 (10:53 +0200)
See also:
http://article.gmane.org/gmane.linux.kernel/1268792

The C compiler will handle (void) as "no arguments" and () as "variadic
function" (equivalent to (...)) which might lead to subtle errors, such
as the one which was fixed with commit 0ea64ae4.

34 files changed:
include/config.h
include/ewmh.h
include/handlers.h
include/ipc.h
include/libi3.h
include/log.h
include/manage.h
include/randr.h
include/sighandler.h
include/tree.h
include/workspace.h
include/x.h
include/xcursor.h
include/xinerama.h
libi3/font.c
src/cfgparse.y
src/commands.c
src/commands_parser.c
src/config.c
src/ewmh.c
src/floating.c
src/handlers.c
src/ipc.c
src/log.c
src/main.c
src/manage.c
src/randr.c
src/sighandler.c
src/tree.c
src/util.c
src/workspace.c
src/x.c
src/xcursor.c
src/xinerama.c

index c1705d4f65d1720ba7fd4d1c888c982a0fcc6f81..e959a2dc6f6f4787dc190b2ca02970bb9ba3ca1f 100644 (file)
@@ -277,7 +277,7 @@ void load_configuration(xcb_connection_t *conn, const char *override_configfile,
  * Translates keysymbols to keycodes for all bindings which use keysyms.
  *
  */
-void translate_keysyms();
+void translate_keysyms(void);
 
 /**
  * Ungrabs all keys, to be called before re-grabbing the keys because of a
index aacd1510f5911f26b65e5a59fb645b526319af97..a786069a337dc9cb8171d00a6ef160231a771e92 100644 (file)
@@ -17,7 +17,7 @@
  * and _NET_NUMBER_OF_DESKTOPS - 1.
  *
  */
-void ewmh_update_current_desktop();
+void ewmh_update_current_desktop(void);
 
 /**
  * Updates _NET_ACTIVE_WINDOW with the currently focused window.
@@ -44,6 +44,6 @@ void ewmh_update_client_list_stacking(xcb_window_t *stack, int num_windows);
  * Set up the EWMH hints on the root window.
  *
  */
-void ewmh_setup_hints();
+void ewmh_setup_hints(void);
 
 #endif
index ebec34cf7e0f68c87b6c847bf84b8e81117dd62e..bcebf9ff0901e38cff023f484a335ca517ad71c2 100644 (file)
@@ -43,7 +43,7 @@ void handle_event(int type, xcb_generic_event_t *event);
  * received from X11
  *
  */
-void property_handlers_init();
+void property_handlers_init(void);
 
 #if 0
 /**
index 9b59fb0163d48abd65c90eee0faaf717794c5abb..af80fa4bd18116bc071cf87f360bd577d850bc8c 100644 (file)
@@ -78,7 +78,7 @@ void ipc_send_event(const char *event, uint32_t message_type, const char *payloa
  * when exiting or restarting only!
  *
  */
-void ipc_shutdown();
+void ipc_shutdown(void);
 
 void dump_node(yajl_gen gen, Con *con, bool inplace_restart);
 
index 7b7cf576a7f31370c9298fecda2a67048c53e8c7..b88bcb67632912eaf4d67c235f4348d1bb318b95 100644 (file)
@@ -197,7 +197,7 @@ void set_font(i3Font *font);
  * Frees the resources taken by the current font.
  *
  */
-void free_font();
+void free_font(void);
 
 /**
  * Converts the given string to UTF-8 from UCS-2 big endian. The return value
index 0eb5574444c06fc8b30356721fd9a44ad3fbeb96..e5e20dc12c00611aff2f36781e04e00647e78e62 100644 (file)
@@ -29,7 +29,7 @@ extern int shmlog_size;
  * XDG_RUNTIME_DIR, see get_process_filename()).
  *
  */
-void init_logging();
+void init_logging(void);
 
 /**
  * Enables the given loglevel.
index 833d614fa063867b1a1673fa430b986a5e7c253a..c16d295f6c3eabd53b0604a2fd1c81a39c80962a 100644 (file)
@@ -27,7 +27,7 @@ void manage_existing_windows(xcb_window_t root);
  * side-effects which are to be expected when continuing to run i3.
  *
  */
-void restore_geometry();
+void restore_geometry(void);
 
 /**
  * Do some sanity checks and then reparent the window.
index 4cefba804ea7b13944a822dfa859885a721775f4..ac527bcc9cc86feb45336686dbd63959646a4a05 100644 (file)
@@ -61,13 +61,13 @@ void init_ws_for_output(Output *output, Con *content);
  * (Re-)queries the outputs via RandR and stores them in the list of outputs.
  *
  */
-void randr_query_outputs();
+void randr_query_outputs(void);
 
 /**
  * Returns the first output which is active.
  *
  */
-Output *get_first_output();
+Output *get_first_output(void);
 
 /**
  * Returns the output with the given name if it is active (!) or NULL.
index 5ffef2a35cc3ef5dccf14b3c9b6c92f4e5e915b2..571070b04230079a1ae606400d1346ae8bb688fc 100644 (file)
@@ -16,6 +16,6 @@
  * Setup signal handlers to safely handle SIGSEGV and SIGFPE
  *
  */
-void setup_signal_handler();
+void setup_signal_handler(void);
 
 #endif
index 81fdbe6acab95374499b138a4d09f2b9eaf18e4d..b9159e3b5b134851ae759ebb0935aadcdcd47230 100644 (file)
@@ -42,20 +42,20 @@ void tree_split(Con *con, orientation_t orientation);
  * Moves focus one level up.
  *
  */
-void level_up();
+void level_up(void);
 
 /**
  * Moves focus one level down.
  *
  */
-void level_down();
+void level_down(void);
 
 /**
  * Renders the tree, that is rendering all outputs using render_con() and
  * pushing the changes to X11 using x_push_changes().
  *
  */
-void tree_render();
+void tree_render(void);
 
 /**
  * Closes the current container using tree_close().
index 8e682bb0298405504799c1652e590a8fbffb839b..1b25b425f4d21b5e4fe0f5e5ab08fd34174f23bf 100644 (file)
@@ -69,31 +69,31 @@ void workspace_show_by_name(const char *num);
  * Returns the next workspace.
  *
  */
-Con* workspace_next();
+Con* workspace_next(void);
 
 /**
  * Returns the previous workspace.
  *
  */
-Con* workspace_prev();
+Con* workspace_prev(void);
 
 /**
  * Returns the next workspace on the same output
  *
  */
-Con* workspace_next_on_output();
+Con* workspace_next_on_output(void);
 
 /**
  * Returns the previous workspace on the same output
  *
  */
-Con* workspace_prev_on_output();
+Con* workspace_prev_on_output(void);
 
 /**
  * Focuses the previously focused workspace.
  *
  */
-void workspace_back_and_forth();
+void workspace_back_and_forth(void);
 
 
 #if 0
index 36c774316605552bf35ba2a687cedecfa915a3ec..cb4a8a964d32823ab74dd1161c1469648ba5ea38 100644 (file)
@@ -108,7 +108,7 @@ void x_set_name(Con *con, const char *name);
  * Sets up i3 specific atoms (I3_SOCKET_PATH and I3_CONFIG_PATH)
  *
  */
-void x_set_i3_atoms();
+void x_set_i3_atoms(void);
 
 /**
  * Set warp_to coordinates.  This will trigger on the next call to
index 2a67bcb75af31929c7438a46cf7a86d8f6a63a42..c341f90ba6b77e6a2bfc028abb13528e2cc96b00 100644 (file)
@@ -20,7 +20,7 @@ enum xcursor_cursor_t {
     XCURSOR_CURSOR_MAX
 };
 
-void xcursor_load_cursors();
+void xcursor_load_cursors(void);
 Cursor xcursor_get_cursor(enum xcursor_cursor_t c);
 int xcursor_get_xcb_cursor(enum xcursor_cursor_t c);
 
index e6304b56bea79f4ab30760a3fb14caadd75ff25f..8c879c07fb7329c38b0c7fe8b2c5419a557d6cd7 100644 (file)
@@ -19,6 +19,6 @@
  * Xinerama information to setup workspaces for each screen.
  *
  */
-void xinerama_init();
+void xinerama_init(void);
 
 #endif
index 4b308efb06eaafbcb6a386bd782afa00dc55b546..0b276b0b3f52c48f9afe5b3011c1dfcf8bb8dbc9 100644 (file)
@@ -87,7 +87,7 @@ void set_font(i3Font *font) {
  * Frees the resources taken by the current font.
  *
  */
-void free_font() {
+void free_font(void) {
     /* Close the font and free the info */
     xcb_close_font(conn, savedFont->id);
     if (savedFont->info)
index 676e0d111181d0c9124a08e7c34e9386e1d63e47..7aeb0a7eb48eb6a37394680db480603b8081ce64 100644 (file)
@@ -51,7 +51,7 @@ void yyerror(const char *error_message) {
     ELOG("\n");
 }
 
-int yywrap() {
+int yywrap(void) {
     return 1;
 }
 
index 8111b54c201bb222ed2e65786408d247e401d555..2df26f6cd83187daef8d1854d7a55bfa13472461 100644 (file)
@@ -108,7 +108,7 @@ static void nagbar_cleanup(EV_P_ ev_cleanup *watcher, int revent) {
 }
 #endif
 
-void cmd_MIGRATION_start_nagbar() {
+void cmd_MIGRATION_start_nagbar(void) {
     if (migration_pid != -1) {
         fprintf(stderr, "i3-nagbar already running.\n");
         return;
index 27a782a2ccf8d8735a55e2d31e78831aec8d72fa..d15fea35569f7e3f2f5796e7639f091c84f3bd9f 100644 (file)
@@ -110,7 +110,7 @@ static char *get_string(const char *identifier) {
     return NULL;
 }
 
-static void clear_stack() {
+static void clear_stack(void) {
     DLOG("clearing stack.\n");
     for (int c = 0; c < 10; c++) {
         if (stack[c].str != NULL)
index de59420a562503ff259b498aa67aca56ec61c160..50ec28234030702fe573d7a8ea858d2bb97b6300 100644 (file)
@@ -81,7 +81,7 @@ Binding *get_binding(uint16_t modifiers, xcb_keycode_t keycode) {
  * Translates keysymbols to keycodes for all bindings which use keysyms.
  *
  */
-void translate_keysyms() {
+void translate_keysyms(void) {
     Binding *bind;
     xcb_keysym_t keysym;
     int col;
index a11d46583d51689df8fadccd6fa4b2321873d651..1c6d918bc30bce445c5c516cbf30fb3b0d38631c 100644 (file)
@@ -16,7 +16,7 @@
  * and _NET_NUMBER_OF_DESKTOPS - 1.
  *
  */
-void ewmh_update_current_desktop() {
+void ewmh_update_current_desktop(void) {
     Con *focused_ws = con_get_workspace(focused);
     Con *output;
     uint32_t idx = 0;
@@ -59,7 +59,7 @@ void ewmh_update_active_window(xcb_window_t window) {
  * desktop applications to place desktop icons appropriately.
  *
  */
-void ewmh_update_workarea() {
+void ewmh_update_workarea(void) {
     int num_workspaces = 0, count = 0;
     Rect last_rect = {0, 0, 0, 0};
     Con *output;
@@ -131,7 +131,7 @@ void ewmh_update_client_list_stacking(xcb_window_t *stack, int num_windows) {
  * Set up the EWMH hints on the root window.
  *
  */
-void ewmh_setup_hints() {
+void ewmh_setup_hints(void) {
     xcb_atom_t supported_atoms[] = {
 #define xmacro(atom) A_ ## atom,
 #include "atoms.xmacro"
index 0b2468477cf41a1b00a2533e2e63121acf2130e8..3b69116988fc8470655e2f24b486486628b1a9c6 100644 (file)
@@ -15,7 +15,7 @@ extern xcb_connection_t *conn;
  * Calculates sum of heights and sum of widths of all currently active outputs
  *
  */
-Rect total_outputs_dimensions() {
+static Rect total_outputs_dimensions(void) {
     Output *output;
     /* Use Rect to encapsulate dimensions, ignoring x/y */
     Rect outputs_dimensions = {0, 0, 0, 0};
index 688cf2f60d614f7d734e0100547c6bb1f9be544b..490bf6eb5edc4f6cbffd4fd76574fef09ba9c762 100644 (file)
@@ -1007,7 +1007,7 @@ static struct property_handler_t property_handlers[] = {
  * received from X11
  *
  */
-void property_handlers_init() {
+void property_handlers_init(void) {
 
     sn_monitor_context_new(sndisplay, conn_screen, startup_monitor_event, NULL, NULL);
 
index 3733034dec5713ae6fabaef1988beadf83010088..40ad868414e9afdd44912bf4ce27bf7dd7a53a82 100644 (file)
--- a/src/ipc.c
+++ b/src/ipc.c
@@ -97,7 +97,7 @@ void ipc_send_event(const char *event, uint32_t message_type, const char *payloa
  * when exiting or restarting only!
  *
  */
-void ipc_shutdown() {
+void ipc_shutdown(void) {
     ipc_client *current;
     while (!TAILQ_EMPTY(&all_clients)) {
         current = TAILQ_FIRST(&all_clients);
index 7b7eca5045e7b22537d71806ccb2249e9672747a..fd98081bf01a92113391971ee2e1eaaca8d8bbbb 100644 (file)
--- a/src/log.c
+++ b/src/log.c
@@ -59,7 +59,7 @@ static int logbuffer_shm;
  * Necessary to print the i3 SHM log in the correct order.
  *
  */
-static void store_log_markers() {
+static void store_log_markers(void) {
     i3_shmlog_header *header = (i3_shmlog_header*)logbuffer;
 
     header->offset_next_write = (logwalk - logbuffer);
@@ -74,7 +74,7 @@ static void store_log_markers() {
  * Will be called twice if --shmlog-size is specified.
  *
  */
-void init_logging() {
+void init_logging(void) {
     if (!errorfilename) {
         if (!(errorfilename = get_process_filename("errorlog")))
             ELOG("Could not initialize errorlog\n");
index c4cffa5cfc3984ff1591480b23e67781aae8c299..0d7246078f06296a1891c8c5f3b1453e584263f9 100644 (file)
@@ -212,7 +212,7 @@ static void xkb_got_event(EV_P_ struct ev_io *w, int revents) {
  * Exit handler which destroys the main_loop. Will trigger cleanup handlers.
  *
  */
-static void i3_exit() {
+static void i3_exit(void) {
 /* We need ev >= 4 for the following code. Since it is not *that* important (it
  * only makes sure that there are no i3-nagbar instances left behind) we still
  * support old systems with libev 3. */
index b0a6aafa5f176b3726b84857efa306854917a8af..ea060d97e9931a847cccbbe96ac0098b3c885c80 100644 (file)
@@ -47,7 +47,7 @@ void manage_existing_windows(xcb_window_t root) {
  * side-effects which are to be expected when continuing to run i3.
  *
  */
-void restore_geometry() {
+void restore_geometry(void) {
     DLOG("Restoring geometry\n");
 
     Con *con;
index 85f0eab3a916153c5eec4075fac99f75b33f6544..73adbf0ea6042327508ff8b619a395fdbf7c66bb 100644 (file)
@@ -60,7 +60,7 @@ Output *get_output_by_name(const char *name) {
  * Returns the first output which is active.
  *
  */
-Output *get_first_output() {
+Output *get_first_output(void) {
     Output *output;
 
     TAILQ_FOREACH(output, &outputs, outputs)
@@ -539,7 +539,7 @@ static void handle_output(xcb_connection_t *conn, xcb_randr_output_t id,
  * (Re-)queries the outputs via RandR and stores them in the list of outputs.
  *
  */
-void randr_query_outputs() {
+void randr_query_outputs(void) {
     Output *output, *other, *first;
     xcb_randr_get_output_primary_cookie_t pcookie;
     xcb_randr_get_screen_resources_current_cookie_t rcookie;
index 4a0c13b5aa1a396f40fa5f29af9c7ff820652aa8..fa608ed89203924fc2bf007b024b7f7ec779752c 100644 (file)
@@ -192,7 +192,7 @@ void handle_signal(int sig, siginfo_t *info, void *data) {
  * Setup signal handlers to safely handle SIGSEGV and SIGFPE
  *
  */
-void setup_signal_handler() {
+void setup_signal_handler(void) {
     struct sigaction action;
 
     action.sa_sigaction = handle_signal;
index 5559908f0738d20afbbed8e9b5f048a7609a4e65..f29369c607eba2d0e64b64d630a0d6ffe78753c2 100644 (file)
@@ -19,7 +19,7 @@ struct all_cons_head all_cons = TAILQ_HEAD_INITIALIZER(all_cons);
  * __i3_scratch will live there.
  *
  */
-static Con *_create___i3() {
+static Con *_create___i3(void) {
     Con *__i3 = con_new(croot, NULL);
     FREE(__i3->name);
     __i3->name = sstrdup("__i3");
@@ -378,7 +378,7 @@ void tree_split(Con *con, orientation_t orientation) {
  * Moves focus one level up.
  *
  */
-void level_up() {
+void level_up(void) {
     /* We cannot go up when we are in fullscreen mode at the moment, that would
      * be totally not intuitive */
     if (focused->fullscreen_mode != CF_NONE) {
@@ -399,7 +399,7 @@ void level_up() {
  * Moves focus one level down.
  *
  */
-void level_down() {
+void level_down(void) {
     /* Go down the focus stack of the current node */
     Con *next = TAILQ_FIRST(&(focused->focus_head));
     if (next == TAILQ_END(&(focused->focus_head))) {
@@ -428,7 +428,7 @@ static void mark_unmapped(Con *con) {
  * pushing the changes to X11 using x_push_changes().
  *
  */
-void tree_render() {
+void tree_render(void) {
     if (croot == NULL)
         return;
 
index 70984f2b7d5bd6422780981f459030428fb18788..d337963eaa093b2c1bd8f7d29684640f4b099bb8 100644 (file)
@@ -222,7 +222,7 @@ char *get_process_filename(const char *prefix) {
 #define y(x, ...) yajl_gen_ ## x (gen, ##__VA_ARGS__)
 #define ystr(str) yajl_gen_string(gen, (unsigned char*)str, strlen(str))
 
-char *store_restart_layout() {
+char *store_restart_layout(void) {
     setlocale(LC_NUMERIC, "C");
 #if YAJL_MAJOR >= 2
     yajl_gen gen = yajl_gen_alloc(NULL);
index 4fc8ba1bda3c421905c7a3b1720cd1cc7e506879..75b8a7b3f3289ce964e96494c48a1518a56a015f 100644 (file)
@@ -393,7 +393,7 @@ void workspace_show_by_name(const char *num) {
  * Focuses the next workspace.
  *
  */
-Con* workspace_next() {
+Con* workspace_next(void) {
     Con *current = con_get_workspace(focused);
     Con *next = NULL;
     Con *output;
@@ -463,7 +463,7 @@ workspace_next_end:
  * Focuses the previous workspace.
  *
  */
-Con* workspace_prev() {
+Con* workspace_prev(void) {
     Con *current = con_get_workspace(focused);
     Con *prev = NULL;
     Con *output;
@@ -535,7 +535,7 @@ workspace_prev_end:
  * Focuses the next workspace on the same output.
  *
  */
-Con* workspace_next_on_output() {
+Con* workspace_next_on_output(void) {
     Con *current = con_get_workspace(focused);
     Con *next = NULL;
     Con *output  = con_get_output(focused);
@@ -590,7 +590,7 @@ workspace_next_on_output_end:
  * Focuses the previous workspace on same output.
  *
  */
-Con* workspace_prev_on_output() {
+Con* workspace_prev_on_output(void) {
     Con *current = con_get_workspace(focused);
     Con *prev = NULL;
     Con *output  = con_get_output(focused);
@@ -646,7 +646,7 @@ workspace_prev_on_output_end:
  * Focuses the previously focused workspace.
  *
  */
-void workspace_back_and_forth() {
+void workspace_back_and_forth(void) {
     if (!previous_workspace_name) {
         DLOG("No previous workspace name set. Not switching.");
         return;
diff --git a/src/x.c b/src/x.c
index f2b53251e42513d16ce468b8c46d5a54e497fdb7..08eb8fee30be8393cf4b752c57aebd7d89a47be1 100644 (file)
--- a/src/x.c
+++ b/src/x.c
@@ -1013,7 +1013,7 @@ void x_set_name(Con *con, const char *name) {
  * Sets up i3 specific atoms (I3_SOCKET_PATH and I3_CONFIG_PATH)
  *
  */
-void x_set_i3_atoms() {
+void x_set_i3_atoms(void) {
     xcb_change_property(conn, XCB_PROP_MODE_REPLACE, root, A_I3_SOCKET_PATH, A_UTF8_STRING, 8,
                         (current_socketpath == NULL ? 0 : strlen(current_socketpath)),
                         current_socketpath);
index f04a3959922612e766fdcd991ee9e839274abc70..058b7ae004c740e1ab7cf6effebdca4ea10fd3d8 100644 (file)
@@ -31,7 +31,7 @@ static Cursor load_cursor(const char *name) {
     return c;
 }
 
-void xcursor_load_cursors() {
+void xcursor_load_cursors(void) {
     cursors[XCURSOR_CURSOR_POINTER] = load_cursor("left_ptr");
     cursors[XCURSOR_CURSOR_RESIZE_HORIZONTAL] = load_cursor("sb_h_double_arrow");
     cursors[XCURSOR_CURSOR_RESIZE_VERTICAL] = load_cursor("sb_v_double_arrow");
index c3ad97c3d7ed62c22ff71412abdc6cb1a5f62850..f377840f22e246ba17cd79c8287822e505981c8a 100644 (file)
@@ -90,7 +90,7 @@ static void query_screens(xcb_connection_t *conn) {
  * information to setup workspaces for each screen.
  *
  */
-void xinerama_init() {
+void xinerama_init(void) {
     if (!xcb_get_extension_data(conn, &xcb_xinerama_id)->present) {
         DLOG("Xinerama extension not found, disabling.\n");
         disable_randr(conn);