wmaker: Cleaned dangerous function prototype usage
authorChristophe CURIS <christophe.curis@free.fr>
Sat, 11 May 2013 22:24:52 +0000 (12 00:24 +0200)
committerCarlos R. Mafra <crmafra@gmail.com>
Sun, 12 May 2013 00:01:20 +0000 (12 01:01 +0100)
- remove extern declaration in source file, use header instead

- add inclusion of header defining the functions of the file to
get the compiler to cross-check them

- marked static the functions that should not be visible ouside
their file

17 files changed:
src/actions.c
src/colormap.c
src/event.c
src/funcs.h
src/geomview.c
src/main.c
src/monitor.c
src/moveres.c
src/osdep_bsd.c
src/osdep_darwin.c
src/osdep_linux.c
src/osdep_stub.c
src/placement.c
src/placement.h
src/properties.c
src/rootmenu.c
src/workspace.c

index 3fb3fe6..17046cf 100644 (file)
@@ -56,7 +56,6 @@ extern Time LastTimestamp;
 extern Time LastFocusChange;
 extern WPreferences wPreferences;
 extern Atom _XA_WM_TAKE_FOCUS;
-extern void ProcessPendingEvents();
 
 static void find_Maximus_geometry(WWindow *wwin, WArea usableArea, int *new_x, int *new_y,
                                  unsigned int *new_width, unsigned int *new_height);
index 3a8906a..121d33b 100644 (file)
 
 #include "WindowMaker.h"
 #include <X11/Xatom.h>
-#include "window.h"
 
+#include "window.h"
 #include "framewin.h"
+#include "colormap.h"
+
+
 void wColormapInstallForWindow(WScreen * scr, WWindow * wwin)
 {
        int i;
index a05d648..692fd8a 100644 (file)
@@ -327,7 +327,6 @@ void DispatchEvent(XEvent * event)
 #define BUFF_SIZE ((sizeof(struct inotify_event) + 64)*5)
 static void handle_inotify_events(int fd, int wd)
 {
-       extern void wDefaultsCheckDomains(void *);
        ssize_t eventQLength, i = 0;
        char buff[BUFF_SIZE] = { 0 };
        /* Check config only once per read of the event queue */
index 848fc40..b996856 100644 (file)
@@ -41,6 +41,7 @@ void StartWindozeCycle(WWindow *wwin, XEvent *event, Bool next, Bool class_only)
 
 void EventLoop(void);
 void DispatchEvent(XEvent *event);
+void ProcessPendingEvents(void);
 WMagicNumber wAddDeathHandler(pid_t pid, WDeathHandler *callback, void *cdata);
 Bool IsDoubleClick(WScreen *scr, XEvent *event);
 
index f067ce4..859cb4c 100644 (file)
@@ -1,6 +1,9 @@
 
 #include <WINGs/WINGsP.h>
 
+#include "geomview.h"
+
+
 typedef struct W_GeometryView {
        W_Class widgetClass;
        WMView *view;
index f1a63b3..b1958e8 100644 (file)
@@ -52,6 +52,7 @@
 #include "session.h"
 #include "shutdown.h"
 #include "dialog.h"
+#include "main.h"
 
 #include <WINGs/WUtil.h>
 
index 9267709..22eaaa9 100644 (file)
@@ -43,7 +43,7 @@
 /****** Global Variables ******/
 extern WPreferences wPreferences;
 
-int showCrashDialog(int sig)
+static int showCrashDialog(int sig)
 {
        int crashAction;
 
index 2d3574e..23045eb 100644 (file)
@@ -37,6 +37,7 @@
 #include "funcs.h"
 #include "actions.h"
 #include "workspace.h"
+#include "placement.h"
 
 #include "geomview.h"
 #include "screen.h"
index dbcdf9e..e92a7c8 100644 (file)
@@ -32,6 +32,7 @@
 #include <WINGs/WUtil.h>
 
 #include "wconfig.h"
+#include "funcs.h"
 
 /*
  * copy argc and argv for an existing process identified by `pid'
index ca38df2..04ece9f 100644 (file)
@@ -9,6 +9,7 @@
 #include <WINGs/WUtil.h>
 
 #include "wconfig.h"
+#include "funcs.h"
 
 /*
  * copy argc and argv for an existing process identified by `pid'
index fd97f59..1336d2f 100644 (file)
@@ -12,6 +12,8 @@
 #include <WINGs/WUtil.h>
 
 #include "wconfig.h"
+#include "funcs.h"
+
 
 #define        RETRY( x )      do {                            \
                                x;                      \
index 79fe9c1..d0349cd 100644 (file)
@@ -4,6 +4,7 @@
 #include <WINGs/WUtil.h>
 
 #include "wconfig.h"
+#include "funcs.h"
 
 Bool GetCommandForPid(int pid, char ***argv, int *argc)
 {
index 5e4a8e7..1e70709 100644 (file)
@@ -38,6 +38,7 @@
 #include "application.h"
 #include "dock.h"
 #include "xinerama.h"
+#include "placement.h"
 
 extern WPreferences wPreferences;
 
@@ -47,10 +48,6 @@ extern WPreferences wPreferences;
 #define Y_ORIGIN WMAX(usableArea.y1,\
     wPreferences.window_place_origin.y)
 
-/* interactive window placement is in moveres.c */
-extern void InteractivePlaceWindow(WWindow *wwin, int *x_ret, int *y_ret,
-                                  unsigned width, unsigned height);
-
 /* Returns True if it is an icon and is in this workspace */
 static Bool
 iconPosition(WCoreWindow *wcore, int sx1, int sy1, int sx2, int sy2,
index e9bfed8..d3455ee 100644 (file)
@@ -33,6 +33,8 @@ int calcIntersectionArea(int x1, int y1, int w1, int h1, int x2, int y2, int w2,
 
 void PlaceWindow(WWindow *wwin, int *x_ret, int *y_ret, unsigned width, unsigned height);
 
+void InteractivePlaceWindow(WWindow * wwin, int *x_ret, int *y_ret, unsigned width, unsigned height);
+
 /* Set the points x and y inside the screen */
 void get_right_position_on_screen(WScreen *scr, int *x, int *y, int size_x, int size_y);
 
index 0442077..5bde912 100644 (file)
@@ -29,6 +29,7 @@
 #include "WindowMaker.h"
 #include "window.h"
 #include "GNUstep.h"
+#include "properties.h"
 
 /* atoms */
 extern Atom _XA_WM_STATE;
index 28bed80..f93e5bf 100644 (file)
@@ -52,6 +52,7 @@
 #include "session.h"
 #include "shutdown.h"
 #include "xmodifier.h"
+#include "rootmenu.h"
 
 #include <WINGs/WUtil.h>
 
index b4cccd0..ab79e68 100644 (file)
@@ -61,7 +61,6 @@
 extern int ignore_wks_change;
 extern WPreferences wPreferences;
 extern XContext wVEdgeContext;
-extern void ProcessPendingEvents();
 extern WShortKey wKeyBindings[WKBD_LAST];
 
 static WMPropList *dWorkspaces = NULL;