WPrefs: Cleaned dangerous function prototype usage
authorChristophe CURIS <christophe.curis@free.fr>
Sat, 11 May 2013 22:24:54 +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

WPrefs.app/Appearance.c
WPrefs.app/Focus.c
WPrefs.app/Menu.c
WPrefs.app/MouseSettings.c
WPrefs.app/WPrefs.c
WPrefs.app/WPrefs.h
WPrefs.app/double.c
WPrefs.app/main.c
WPrefs.app/xmodifier.c

index 9c3039c..65865e9 100644 (file)
@@ -2036,7 +2036,7 @@ static void prepareForClose(_Panel * panel)
        WMSynchronizeUserDefaults(udb);
 }
 
-Panel *InitAppearance(WMScreen * scr, WMWindow * win)
+Panel *InitAppearance(WMScreen *scr, WMWidget *parent)
 {
        _Panel *panel;
 
@@ -2046,7 +2046,7 @@ Panel *InitAppearance(WMScreen * scr, WMWindow * win)
 
        panel->description = _("Background texture configuration for windows,\n" "menus and icons.");
 
-       panel->parent = win;
+       panel->parent = parent;
 
        panel->callbacks.createWidgets = createPanel;
        panel->callbacks.updateDomain = storeData;
index d664cfa..9567dde 100644 (file)
@@ -330,7 +330,7 @@ static void createPanel(Panel * p)
        showData(panel);
 }
 
-Panel *InitFocus(WMScreen * scr, WMWindow * win)
+Panel *InitFocus(WMScreen *scr, WMWidget *parent)
 {
        _Panel *panel;
 
@@ -338,7 +338,7 @@ Panel *InitFocus(WMScreen * scr, WMWindow * win)
 
        panel->sectionName = _("Window Focus Preferences");
        panel->description = _("Keyboard focus switching policy and related options.");
-       panel->parent = win;
+       panel->parent = parent;
 
        panel->callbacks.createWidgets = createPanel;
        panel->callbacks.updateDomain = storeData;
index 1f00d53..752bd8f 100644 (file)
@@ -179,7 +179,6 @@ static Bool shouldRemoveItem(struct WEditMenuDelegate *delegate, WEditMenu * men
 
 static void freeItemData(ItemData * data);
 
-extern char *capture_shortcut(Display *dpy, Bool *capturing, Bool convert_case);
 
 static WEditMenuDelegate menuDelegate = {
        NULL,
index 2c836ae..bebf9f5 100644 (file)
@@ -158,7 +158,6 @@ static void doubleClick(WMWidget * w, void *data)
 {
        _Panel *panel = (_Panel *) data;
        int i;
-       extern _WINGsConfiguration WINGsConfiguration;
        char buffer[32];
 
        for (i = 0; i < 5; i++) {
index 3acbd39..6d74409 100644 (file)
 #include "WPrefs.h"
 #include <assert.h>
 
-extern Panel *InitWindowHandling(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitKeyboardSettings(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitMouseSettings(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitKeyboardShortcuts(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitWorkspace(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitDocks(WMScreen *scr, WMWidget *parent);
-
-extern Panel *InitFocus(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitPreferences(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitFont(WMScreen * scr, WMWidget * parent);
-extern Panel *InitFontSimple(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitConfigurations(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitPaths(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitMenu(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitExpert(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitMenuPreferences(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitIcons(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitThemes(WMScreen * scr, WMWidget * parent);
-
-extern Panel *InitAppearance(WMScreen * scr, WMWidget * parent);
 
 #define ICON_TITLE_FONT "sans serif:pixelsize=9"
 #define ICON_TITLE_VFONT "sans serif:pixelsize=9:weight=100"
@@ -104,7 +70,7 @@ static void savePanelData(Panel * panel);
 
 static void prepareForClose(void);
 
-void quit(WMWidget * w, void *data)
+static void quit(WMWidget *w, void *data)
 {
        prepareForClose();
 
@@ -200,7 +166,7 @@ static void prepareForClose(void)
        }
 }
 
-void toggleBalloons(WMWidget * w, void *data)
+static void toggleBalloons(WMWidget *w, void *data)
 {
        WMUserDefaults *udb = WMGetStandardUserDefaults();
        Bool flag;
index c640061..021bf3b 100644 (file)
@@ -111,6 +111,11 @@ void SetBoolForKey(Bool value, const char *defaultName);
 void SetSpeedForKey(int speed, const char *defaultName);
 
 
+/* ---[ KeyboardShortcuts.c ] -------------------------------------------- */
+
+char *capture_shortcut(Display *dpy, Bool *capturing, Bool convert_case);
+
+
 /* ---[ double.c ] ------------------------------------------------------- */
 typedef struct W_DoubleTest DoubleTest;
 
@@ -125,6 +130,28 @@ void AddDeadChildHandler(pid_t pid, void (*handler)(void*), void *data);
 int ModifierFromKey(Display * dpy, const char *key);
 
 
+/* ---[ Panel Initializers ]---------------------------------------------- */
+
+void Initialize(WMScreen *scr);
+
+/* in alphabetical order - in case you'd want to add one */
+Panel *InitAppearance(WMScreen *scr, WMWidget *parent);
+Panel *InitConfigurations(WMScreen *scr, WMWidget *parent);
+Panel *InitDocks(WMScreen *scr, WMWidget *parent);
+Panel *InitExpert(WMScreen *scr, WMWidget *parent);
+Panel *InitFocus(WMScreen *scr, WMWidget *parent);
+Panel *InitFontSimple(WMScreen *scr, WMWidget *parent);
+Panel *InitIcons(WMScreen *scr, WMWidget *parent);
+Panel *InitKeyboardShortcuts(WMScreen *scr, WMWidget *parent);
+Panel *InitMenu(WMScreen *scr, WMWidget *parent);
+Panel *InitMenuPreferences(WMScreen *scr, WMWidget *parent);
+Panel *InitMouseSettings(WMScreen *scr, WMWidget *parent);
+Panel *InitPaths(WMScreen *scr, WMWidget *parent);
+Panel *InitPreferences(WMScreen *scr, WMWidget *parent);
+Panel *InitWindowHandling(WMScreen *scr, WMWidget *parent);
+Panel *InitWorkspace(WMScreen *scr, WMWidget *parent);
+
+
 #define FRAME_TOP      105
 #define FRAME_LEFT     -2
 #define FRAME_WIDTH    524
index 715e78c..56e33f8 100644 (file)
@@ -133,7 +133,6 @@ static void deactivate(void *data)
 static void handleActionEvents(XEvent * event, void *data)
 {
        _DoubleTest *dPtr = (_DoubleTest *) data;
-       extern _WINGsConfiguration WINGsConfiguration;
 
        switch (event->type) {
        case ButtonPress:
index 6e9cd9e..561113f 100644 (file)
@@ -29,7 +29,6 @@
 
 char *NOptionValueChanged = "NOptionValueChanged";
 
-extern void Initialize(WMScreen * scr);
 
 #define MAX_DEATHS     64
 
@@ -42,7 +41,7 @@ struct {
 static pid_t DeadChildren[MAX_DEATHS];
 static int DeadChildrenCount = 0;
 
-void wAbort(Bool foo)
+static void wAbort(Bool foo)
 {
        exit(1);
 }
index c7ac1d5..d0b1a22 100644 (file)
@@ -39,6 +39,8 @@ Perpetrator: Sudish Joseph <sj@eng.mindspring.net>, Sept. 1997. */
 
 #include <WINGs/WUtil.h>
 
+#include "WPrefs.h"
+
 /************************************************************************/
 /*                            keymap handling                           */
 /************************************************************************/