Indent. Remove unused function. Comment comments.

SVN revision: 9396

src/E.h

@@ -1676,7 +1676,7 @@ void                AddToFamily(Window win);
 EWin               *AddInternalToFamily(Window win, char *bname, int type,
 					void *ptr,
 					void (*init) (EWin * ewin, void *ptr));
-void                CalcEwinSizes(EWin *ewin);
+void                CalcEwinSizes(EWin * ewin);
 void                HonorIclass(char *s, int id);
 void                SyncBorderToEwin(EWin * ewin);
 void                UpdateBorderInfo(EWin * ewin);
@@ -2236,7 +2236,7 @@ void                FreePmapMask(PmapMask * pmm);
 void                IB_Animate(char iconify, EWin * from, EWin * to);
 void                IconifyEwin(EWin * ewin);
 void                DeIconifyEwin(EWin * ewin);
-void                MakeIcon(EWin *ewin);
+void                MakeIcon(EWin * ewin);
 void                RemoveMiniIcon(EWin * ewin);
 Iconbox            *IconboxCreate(char *name);
 void                IconboxDestroy(Iconbox * ib);

src/borders.c

@@ -193,12 +193,7 @@ GetContextEwin(void)
    if (mode.focuswin)
       EDBUG_RETURN(mode.focuswin);
 
-   /* FIXME: Is this semicolin supposed to be here in this if statement? */
-   /* until someone says one way or another its commented out --Mandrake */
-   /* if (mode.mouse_over_win); */
    EDBUG_RETURN(mode.mouse_over_win);
-
-   return NULL;
 }
 
 void

src/desktops.c

@@ -562,7 +562,7 @@ SetBackgroundTo(Window win, Background * dsk, char setbg)
 
    if (!dpmap)
      {
-	unsigned int                 w, h, x, y;
+	unsigned int        w, h, x, y;
 	char                hasbg, hasfg;
 	Pixmap              pmap, mask;
 	ColorModifierClass *cm;

src/fx.c

@@ -82,7 +82,7 @@ static FXHandler    fx_handlers[] = {
 static FXHandler   *
 FX_Find(const char *name)
 {
-   unsigned int                 i;
+   unsigned int        i;
 
    for (i = 0; i < N_FX_HANDLERS; i++)
       if (!strcmp(fx_handlers[i].name, name))
@@ -138,7 +138,7 @@ FX_Op(const char *name, int fx_op)
 static void
 FX_Activate(char *effect)
 {
-   unsigned int                 i;
+   unsigned int        i;
 
    for (i = 0; i < N_FX_HANDLERS; i++)
      {
@@ -158,7 +158,7 @@ FX_Activate(char *effect)
 void
 FX_DeskChange(void)
 {
-   unsigned int                 i;
+   unsigned int        i;
 
    for (i = 0; i < N_FX_HANDLERS; i++)
      {
@@ -173,7 +173,7 @@ FX_DeskChange(void)
 void
 FX_Pause(void)
 {
-   unsigned int                 i;
+   unsigned int        i;
 
    for (i = 0; i < N_FX_HANDLERS; i++)
      {
@@ -198,7 +198,7 @@ FX_Pause(void)
 char              **
 FX_Active(int *num)
 {
-   unsigned int                 i;
+   unsigned int        i;
    char              **list = NULL;
 
    *num = 0;
@@ -218,7 +218,7 @@ FX_Active(int *num)
 int
 FX_IsOn(char *effect)
 {
-   unsigned int                 i;
+   unsigned int        i;
 
    for (i = 0; i < N_FX_HANDLERS; i++)
      {

src/handlers.c

@@ -127,7 +127,7 @@ SignalsSetup(void)
       SIGHUP, SIGINT, SIGQUIT, SIGILL, SIGABRT, SIGFPE, SIGSEGV, SIGPIPE,
       SIGALRM, SIGTERM, SIGUSR1, SIGUSR2, SIGCHLD, SIGBUS
    };
-   unsigned int                 i;
+   unsigned int        i;
    struct sigaction    sa;
 
    EDBUG(6, "SignalsSetup");

src/ipc.c

@@ -4106,11 +4106,12 @@ IPC_FocusMode(char *params, Client * c)
 static void
 IPC_ShowIcons(char *params, Client * c)
 {
-    /* Doesn't look like this function is doing anything, but it used to 
-     * if I recall correctly --Mandrake */
-    return;
-    params = NULL;
-    c = NULL;
+   /* Doesn't look like this function is doing anything, but it used to 
+    * if I recall correctly --Mandrake
+    * If it did anything useful, it was before 0.16.5 /Kim */
+   return;
+   params = NULL;
+   c = NULL;
 }
 
 static void

src/menus.c

@@ -1895,7 +1895,7 @@ MenuCreateMoveToDesktop(char *name, MenuStyle * ms)
 }
 #endif
 
-static Menu               *
+static Menu        *
 MenuCreateFromGroups(char *name, MenuStyle * ms)
 {
    Menu               *m, *mm;

src/pager.c

@@ -287,11 +287,12 @@ PagerMoveResize(EWin * ewin, int resize)
 static void
 PagerRefresh(EWin * ewin)
 {
-    /* This doesn't do anything anymore apparently
-     * --Mandrake
-     */
-    return;
-    ewin = NULL;
+   /* This doesn't do anything anymore apparently
+    * --Mandrake
+    * This is new code.
+    * It can be removed but I have kept it around, just in case /Kim */
+   return;
+   ewin = NULL;
 }
 
 static void
@@ -409,23 +410,6 @@ PagerDestroy(Pager * p)
    Efree(p);
 }
 
-#if 0
-/*
- e  Doesn't look like this is ever used, commented out for now
- * --Mandrake
- */
-static void
-PagerOnUnmap(Pager * p)
-{
-   PagerHideHi(p);
-   if (p == mode.context_pager)
-     {
-	mode.context_pager = NULL;
-	mode.mode = MODE_NONE;
-     }
-}
-#endif
-
 Pager             **
 PagersForDesktop(int d, int *num)
 {

src/warp.c

@@ -47,7 +47,7 @@ static Window       warpFocusTitleWindow = 0;
 static int          warptitles_num = 0;
 static Window      *warptitles = NULL;
 static EWin       **warptitles_ewin = NULL;
-static unsigned int          warpFocusKey = 0;
+static unsigned int warpFocusKey = 0;
 
 int
 WarpFocusHandleEvent(XEvent * ev)