From 3157e4be1ec7f2a5ac61ca45b232cd07c5e30483 Mon Sep 17 00:00:00 2001
From: mc78 <mc78@outlook.de>
Date: Mon, 22 May 2017 23:07:31 +0200
Subject: [PATCH] added empty lines and spaces according to olivers comments.
 Made local functions static.

---
 src/cl65/main.c | 55 +++++++++++++++++++++++++++++++++++--------------
 1 file changed, 40 insertions(+), 15 deletions(-)

diff --git a/src/cl65/main.c b/src/cl65/main.c
index e428c96ac..6926199f9 100644
--- a/src/cl65/main.c
+++ b/src/cl65/main.c
@@ -156,22 +156,36 @@ static char* TargetLib  = 0;
 #  endif
 #endif
 
+
+
 /*****************************************************************************/
 /*                        Credential functions                               */
 /*****************************************************************************/
-void DisableAssembling(void){
+
+
+
+static void DisableAssembling (void)
+{
     DoAssemble = 0;
 }
 
-void DisableLinking(void){
+
+
+static void DisableLinking (void)
+{
     DoLink = 0;
 }
 
-void DisableAssemblingAndLinking(void){
+
+
+static void DisableAssemblingAndLinking (void)
+{
     DisableAssembling();
     DisableLinking();
 }
 
+
+
 /*****************************************************************************/
 /*                        Command structure handling                         */
 /*****************************************************************************/
@@ -1427,22 +1441,33 @@ int main (int argc, char* argv [])
                     break;
                 
                 case 'E':
-                    /*Forward -E to compiler*/
+                    /* Forward -E to compiler */
                     CmdAddArg (&CC65, Arg);  
                     DisableAssemblingAndLinking();
                     break;
+                    
                 case 'W':
-                    if (Arg[2] == 'a' && Arg[3] == '\0') {
-                        /* -Wa: Pass options to assembler */
-                        OptAsmArgs (Arg, GetArg (&I, 3));
-                    } else if (Arg[2] == 'c' && Arg[3] == '\0') {
-                        /* -Wc: Pass options to compiler */
-                        /* Remember -Wc sub arguments in cc65 arg struct */ 
-                        OptCCArgs (Arg, GetArg (&I, 3));
-                    } else if (Arg[2] == 'l' && Arg[3] == '\0') {
-                        /* -Wl: Pass options to linker */
-                        OptLdArgs (Arg, GetArg (&I, 3));
-                    } else {
+                    /* avoid && with'\0' in if clauses */
+                    if (Arg[3] == '\0') {
+                        switch (Arg[2]) {
+                        case 'a':
+                            /* -Wa: Pass options to assembler */
+                            OptAsmArgs (Arg, GetArg (&I, 3));
+                            break;
+                        case 'c':
+                            /* -Wc: Pass options to compiler 
+                            ** Remember -Wc sub arguments in cc65 arg struct 
+                            */
+                            OptCCArgs (Arg, GetArg (&I, 3));
+                            break;
+                        case 'l':
+                            /* -Wl: Pass options to linker */
+                            OptLdArgs (Arg, GetArg (&I, 3));
+                            break;
+                        default:
+                            break;
+                       }
+                    }else {
                         /* Anything else: Suppress warnings (compiler) */
                         CmdAddArg2 (&CC65, "-W", GetArg (&I, 2));
                     }
-- 
2.39.5