]> git.sur5r.net Git - cc65/blobdiff - src/cl65/main.c
The assembler options -l and --listing will now take the name of the listing
[cc65] / src / cl65 / main.c
index bd3a3064bb7275d184ce544f0a1c996fd8605d38..cf22adb196502c620ce3cea301bc7448d4b6ac1b 100644 (file)
@@ -6,7 +6,7 @@
 /*                                                                          */
 /*                                                                          */
 /*                                                                          */
-/* (C) 1999-2009, Ullrich von Bassewitz                                      */
+/* (C) 1999-2011, Ullrich von Bassewitz                                      */
 /*                Roemerstrasse 52                                           */
 /*                D-70794 Filderstadt                                        */
 /* EMail:         uz@cc65.org                                                */
@@ -98,8 +98,8 @@ static CmdDesc LD65 = { 0, 0, 0, 0, 0, 0, 0 };
 static CmdDesc GRC  = { 0, 0, 0, 0, 0, 0, 0 };
 
 /* Variables controlling the steps we're doing */
-static int DontLink    = 0;
-static int DontAssemble = 0;
+static int DoLink       = 1;
+static int DoAssemble   = 1;
 
 /* The name of the output file, NULL if none given */
 static const char* OutputName = 0;
@@ -112,6 +112,10 @@ static const char* LinkerConfig = 0;
  */
 static const char* FirstInput = 0;
 
+/* The names of the files for dependency generation */
+static const char* DepName = 0;
+static const char* FullDepName = 0;
+
 /* Remember if we should link a module */
 static int Module = 0;
 
@@ -382,23 +386,6 @@ static void Link (void)
 {
     unsigned I;
 
-    /* If we have a linker config file given, add it to the command line.
-     * Otherwise pass the target to the linker if we have one.
-     */
-    if (LinkerConfig) {
-        if (Module) {
-            Error ("Cannot use -C and --module together");
-        }
-               CmdAddArg2 (&LD65, "-C", LinkerConfig);
-    } else if (Module) {
-        CmdSetTarget (&LD65, TGT_MODULE);
-    } else {
-       CmdSetTarget (&LD65, Target);
-    }
-
-    /* Determine which target libraries are needed */
-    SetTargetFiles ();
-
     /* Since linking is always the final step, if we have an output file name
      * given, set it here. If we don't have an explicit output name given,
      * try to build one from the name of the first input file.
@@ -416,6 +403,23 @@ static void Link (void)
 
     }
 
+    /* If we have a linker config file given, add it to the command line.
+     * Otherwise pass the target to the linker if we have one.
+     */
+    if (LinkerConfig) {
+        if (Module) {
+            Error ("Cannot use -C and --module together");
+        }
+               CmdAddArg2 (&LD65, "-C", LinkerConfig);
+    } else if (Module) {
+        CmdSetTarget (&LD65, TGT_MODULE);
+    } else {
+       CmdSetTarget (&LD65, Target);
+    }
+
+    /* Determine which target libraries are needed */
+    SetTargetFiles ();
+
     /* Add all object files as parameters */
     for (I = 0; I < LD65.FileCount; ++I) {
        CmdAddArg (&LD65, LD65.Files [I]);
@@ -435,28 +439,29 @@ static void Link (void)
 
 
 
-static void Assemble (const char* File)
-/* Assemble the given file */
+static void AssembleFile (const char* File, unsigned ArgCount)
+/* Common routine to assemble a file. Will be called by Assemble() and
+ * AssembleIntermediate(). Adds options common for both routines and
+ * assembles the file. Will remove excess arguments after assembly.
+ */
 {
-    /* Remember the current assembler argument count */
-    unsigned ArgCount = CA65.ArgCount;
-
     /* Set the target system */
     CmdSetTarget (&CA65, Target);
 
-    /* If we won't link, this is the final step. In this case, set the
-     * output name.
-     */
-    if (DontLink && OutputName) {
-       CmdSetOutput (&CA65, OutputName);
+    /* Check if this is the last processing step */
+    if (DoLink) {
+        /* We're linking later. Add the output file of the assembly
+         * the the file list of the linker. The name of the output
+         * file is that of the input file with ".s" replaced by ".o".
+         */
+       char* ObjName = MakeFilename (File, ".o");
+       CmdAddFile (&LD65, ObjName);
+       xfree (ObjName);
     } else {
-       /* The object file name will be the name of the source file
-        * with .s replaced by ".o". Add this file to the list of
-        * linker files.
-        */
-       char* ObjName = MakeFilename (File, ".o");
-       CmdAddFile (&LD65, ObjName);
-       xfree (ObjName);
+        /* This is the final step. If an output name is given, set it */
+        if (OutputName) {
+            CmdSetOutput (&CA65, OutputName);
+        }
     }
 
     /* Add the file as argument for the assembler */
@@ -474,27 +479,90 @@ static void Assemble (const char* File)
 
 
 
+static void AssembleIntermediate (const char* SourceFile)
+/* Assemble an intermediate file which was generated by a previous processing
+ * step with SourceFile as input. The -dep options won't be added and
+ * the intermediate assembler file is removed after assembly.
+ */
+{
+    /* Generate the name of the assembler output file from the source file
+     * name. It's the same name with the extension replaced by ".s"
+     */
+    char* AsmName = MakeFilename (SourceFile, ".s");
+
+    /* Assemble the intermediate assembler file */
+    AssembleFile (AsmName, CA65.ArgCount);
+
+    /* Remove the input file */
+    if (remove (AsmName) < 0) {
+        Warning ("Cannot remove temporary file `%s': %s",
+                 AsmName, strerror (errno));
+    }
+
+    /* Free the assembler file name which was allocated from the heap */
+    xfree (AsmName);
+}
+
+
+
+static void Assemble (const char* File)
+/* Assemble the given file */
+{
+    /* Remember the current assembler argument count */
+    unsigned ArgCount = CA65.ArgCount;
+
+    /* We aren't assembling an intermediate file, but one requested by the
+     * user. So add a few options here if they were given on the command
+     * line.
+     */
+    if (DepName && *DepName) {
+        CmdAddArg2 (&CA65, "--create-dep", DepName);
+    }
+    if (FullDepName && *FullDepName) {
+        CmdAddArg2 (&CA65, "--create-full-dep", FullDepName);
+    }
+
+    /* Use the common routine */
+    AssembleFile (File, ArgCount);
+}
+
+
+
 static void Compile (const char* File)
 /* Compile the given file */
 {
-    char* AsmName = 0;
-
     /* Remember the current compiler argument count */
     unsigned ArgCount = CC65.ArgCount;
 
     /* Set the target system */
     CmdSetTarget (&CC65, Target);
 
-    /* If we won't link, this is the final step. In this case, set the
-     * output name.
-     */
-    if (DontAssemble && OutputName) {
-       CmdSetOutput (&CC65, OutputName);
+    /* Check if this is the final step */
+    if (DoAssemble) {
+        /* We will assemble this file later. If a dependency file is to be
+         * generated, set the dependency target to be the final object file,
+         * not the intermediate assembler file. But beware: There may be an
+         * output name specified for the assembler.
+         */
+        if (DepName || FullDepName) {
+            /* Was an output name for the assembler specified? */
+            if (!DoLink && OutputName) {
+                /* Use this name as the dependency target */
+                CmdAddArg2 (&CC65, "--dep-target", OutputName);
+            } else {
+                /* Use the object file name as the dependency target */
+                char* ObjName = MakeFilename (File, ".o");
+                CmdAddArg2 (&CC65, "--dep-target", ObjName);
+                xfree (ObjName);
+            }
+        }
     } else {
-       /* The assembler file name will be the name of the source file
-        * with .c replaced by ".s".
-        */
-       AsmName = MakeFilename (File, ".s");
+        /* If we won't assemble, this is the final step. In this case, set
+         * the output name if it was given.
+         */
+        if (OutputName) {
+            CmdSetOutput (&CC65, OutputName);
+        }
     }
 
     /* Add the file as argument for the compiler */
@@ -512,13 +580,9 @@ static void Compile (const char* File)
     /* If this is not the final step, assemble the generated file, then
      * remove it
      */
-    if (!DontAssemble) {
-       Assemble (AsmName);
-       if (remove (AsmName) < 0) {
-           Warning ("Cannot remove temporary file `%s': %s",
-                    AsmName, strerror (errno));
-       }
-       xfree (AsmName);
+    if (DoAssemble) {
+        /* Assemble the intermediate file and remove it */
+       AssembleIntermediate (File);
     }
 }
 
@@ -527,16 +591,9 @@ static void Compile (const char* File)
 static void CompileRes (const char* File)
 /* Compile the given geos resource file */
 {
-    char* AsmName = 0;
-
     /* Remember the current assembler argument count */
     unsigned ArgCount = GRC.ArgCount;
 
-    /* The assembler file name will be the name of the source file
-     * with .grc replaced by ".s".
-     */
-    AsmName = MakeFilename (File, ".s");
-
     /* Add the file as argument for the resource compiler */
     CmdAddArg (&GRC, File);
 
@@ -552,16 +609,10 @@ static void CompileRes (const char* File)
     /* If this is not the final step, assemble the generated file, then
      * remove it
      */
-    if (!DontAssemble) {
-       Assemble (AsmName);
-       if (remove (AsmName) < 0) {
-           Warning ("Cannot remove temporary file `%s': %s",
-                    AsmName, strerror (errno));
-       }
+    if (DoAssemble) {
+        /* Assemble the intermediate file and remove it */
+       AssembleIntermediate (File);
     }
-
-    /* Free the assembler file name which was allocated from the heap */
-    xfree (AsmName);
 }
 
 
@@ -569,21 +620,14 @@ static void CompileRes (const char* File)
 static void ConvertO65 (const char* File)
 /* Convert an o65 object file into an assembler file */
 {
-    char* AsmName = 0;
-
     /* Remember the current converter argument count */
     unsigned ArgCount = CO65.ArgCount;
 
-    /* If we won't link, this is the final step. In this case, set the
+    /* If we won't assemble, this is the final step. In this case, set the
      * output name.
      */
-    if (DontAssemble && OutputName) {
-       CmdSetOutput (&CO65, OutputName);
-    } else {
-       /* The assembler file name will be the name of the source file
-        * with .c replaced by ".s".
-        */
-       AsmName = MakeFilename (File, ".s");
+    if (!DoAssemble && OutputName) {
+               CmdSetOutput (&CO65, OutputName);
     }
 
     /* Add the file as argument for the object file converter */
@@ -601,16 +645,10 @@ static void ConvertO65 (const char* File)
     /* If this is not the final step, assemble the generated file, then
      * remove it
      */
-    if (!DontAssemble) {
-       Assemble (AsmName);
-       if (remove (AsmName) < 0) {
-           Warning ("Cannot remove temporary file `%s': %s",
-                    AsmName, strerror (errno));
-       }
+    if (DoAssemble) {
+        /* Assemble the intermediate file and remove it */
+       AssembleIntermediate (File);
     }
-
-    /* Free the assembler file name which was allocated from the heap */
-    xfree (AsmName);
 }
 
 
@@ -626,85 +664,87 @@ static void Usage (void)
 {
     printf ("Usage: %s [options] file [...]\n"
             "Short options:\n"
-            "  -c\t\t\tCompile and assemble but don't link\n"
-            "  -d\t\t\tDebug mode\n"
-            "  -g\t\t\tAdd debug info\n"
-            "  -h\t\t\tHelp (this text)\n"
-            "  -l\t\t\tCreate an assembler listing\n"
-            "  -m name\t\tCreate a map file\n"
-            "  -mm model\t\tSet the memory model\n"
-            "  -o name\t\tName the output file\n"
-            "  -r\t\t\tEnable register variables\n"
-            "  -t sys\t\tSet the target system\n"
-            "  -u sym\t\tForce an import of symbol `sym'\n"
-            "  -v\t\t\tVerbose mode\n"
-            "  -vm\t\t\tVerbose map file\n"
-            "  -C name\t\tUse linker config file\n"
-            "  -Cl\t\t\tMake local variables static\n"
-            "  -D sym[=defn]\t\tDefine a preprocessor symbol\n"
-            "  -I dir\t\tSet a compiler include directory path\n"
-            "  -L path\t\tSpecify a library search path\n"
-            "  -Ln name\t\tCreate a VICE label file\n"
-            "  -O\t\t\tOptimize code\n"
-            "  -Oi\t\t\tOptimize code, inline functions\n"
-            "  -Or\t\t\tOptimize code, honour the register keyword\n"
-            "  -Os\t\t\tOptimize code, inline known C funtions\n"
-            "  -S\t\t\tCompile but don't assemble and link\n"
-            "  -T\t\t\tInclude source as comment\n"
-            "  -V\t\t\tPrint the version number\n"
-            "  -W\t\t\tSuppress warnings\n"
-            "  -Wa options\t\tPass options to the assembler\n"
-            "  -Wl options\t\tPass options to the linker\n"
+            "  -c\t\t\t\tCompile and assemble but don't link\n"
+            "  -d\t\t\t\tDebug mode\n"
+            "  -g\t\t\t\tAdd debug info\n"
+            "  -h\t\t\t\tHelp (this text)\n"
+            "  -l name\t\t\tCreate an assembler listing file\n"
+            "  -m name\t\t\tCreate a map file\n"
+            "  -mm model\t\t\tSet the memory model\n"
+            "  -o name\t\t\tName the output file\n"
+            "  -r\t\t\t\tEnable register variables\n"
+            "  -t sys\t\t\tSet the target system\n"
+            "  -u sym\t\t\tForce an import of symbol `sym'\n"
+            "  -v\t\t\t\tVerbose mode\n"
+            "  -vm\t\t\t\tVerbose map file\n"
+            "  -C name\t\t\tUse linker config file\n"
+            "  -Cl\t\t\t\tMake local variables static\n"
+            "  -D sym[=defn]\t\t\tDefine a preprocessor symbol\n"
+            "  -I dir\t\t\tSet a compiler include directory path\n"
+            "  -L path\t\t\tSpecify a library search path\n"
+            "  -Ln name\t\t\tCreate a VICE label file\n"
+            "  -O\t\t\t\tOptimize code\n"
+            "  -Oi\t\t\t\tOptimize code, inline functions\n"
+            "  -Or\t\t\t\tOptimize code, honour the register keyword\n"
+            "  -Os\t\t\t\tOptimize code, inline known C funtions\n"
+            "  -S\t\t\t\tCompile but don't assemble and link\n"
+            "  -T\t\t\t\tInclude source as comment\n"
+            "  -V\t\t\t\tPrint the version number\n"
+            "  -W\t\t\t\tSuppress warnings\n"
+            "  -Wa options\t\t\tPass options to the assembler\n"
+            "  -Wl options\t\t\tPass options to the linker\n"
             "\n"
             "Long options:\n"
-            "  --add-source\t\tInclude source as comment\n"
-            "  --asm-args options\tPass options to the assembler\n"
-            "  --asm-define sym[=v]\tDefine an assembler symbol\n"
-            "  --asm-include-dir dir\tSet an assembler include directory\n"
-            "  --bss-label name\tDefine and export a BSS segment label\n"
-            "  --bss-name seg\tSet the name of the BSS segment\n"
-            "  --cfg-path path\tSpecify a config file search path\n"
-            "  --check-stack\t\tGenerate stack overflow checks\n"
-            "  --code-label name\tDefine and export a CODE segment label\n"
-            "  --code-name seg\tSet the name of the CODE segment\n"
-            "  --codesize x\t\tAccept larger code by factor x\n"
-            "  --config name\t\tUse linker config file\n"
-            "  --cpu type\t\tSet cpu type\n"
-            "  --create-dep\t\tCreate a make dependency file\n"
-            "  --data-label name\tDefine and export a DATA segment label\n"
-            "  --data-name seg\tSet the name of the DATA segment\n"
-            "  --debug\t\tDebug mode\n"
-            "  --debug-info\t\tAdd debug info\n"
-            "  --feature name\tSet an emulation feature\n"
-            "  --force-import sym\tForce an import of symbol `sym'\n"
-            "  --forget-inc-paths\tForget include search paths (compiler)\n"
-            "  --help\t\tHelp (this text)\n"
-            "  --include-dir dir\tSet a compiler include directory path\n"
-            "  --ld-args options\tPass options to the linker\n"
-            "  --lib file\t\tLink this library\n"
-            "  --lib-path path\tSpecify a library search path\n"
-            "  --list-targets\tList all available targets\n"
-            "  --listing\t\tCreate an assembler listing\n"
-            "  --list-bytes n\tNumber of bytes per assembler listing line\n"
-            "  --mapfile name\tCreate a map file\n"
-            "  --memory-model model\tSet the memory model\n"
-            "  --module\t\tLink as a module\n"
-            "  --module-id id\tSpecify a module id for the linker\n"
-            "  --o65-model model\tOverride the o65 model\n"
-            "  --obj file\t\tLink this object file\n"
-            "  --obj-path path\tSpecify an object file search path\n"
-            "  --register-space b\tSet space available for register variables\n"
-            "  --register-vars\tEnable register variables\n"
-            "  --rodata-name seg\tSet the name of the RODATA segment\n"
-            "  --signed-chars\tDefault characters are signed\n"
-            "  --standard std\tLanguage standard (c89, c99, cc65)\n"
-            "  --start-addr addr\tSet the default start address\n"
-            "  --static-locals\tMake local variables static\n"
-            "  --target sys\t\tSet the target system\n"
-            "  --version\t\tPrint the version number\n"
-            "  --verbose\t\tVerbose mode\n"
-            "  --zeropage-label name\tDefine and export a ZEROPAGE segment label\n"
-            "  --zeropage-name seg\tSet the name of the ZEROPAGE segment\n",
+            "  --add-source\t\t\tInclude source as comment\n"
+            "  --asm-args options\t\tPass options to the assembler\n"
+            "  --asm-define sym[=v]\t\tDefine an assembler symbol\n"
+            "  --asm-include-dir dir\t\tSet an assembler include directory\n"
+            "  --bin-include-dir dir\t\tSet an assembker binary include directory\n"
+            "  --bss-label name\t\tDefine and export a BSS segment label\n"
+            "  --bss-name seg\t\tSet the name of the BSS segment\n"
+            "  --cfg-path path\t\tSpecify a config file search path\n"
+            "  --check-stack\t\t\tGenerate stack overflow checks\n"
+            "  --code-label name\t\tDefine and export a CODE segment label\n"
+            "  --code-name seg\t\tSet the name of the CODE segment\n"
+            "  --codesize x\t\t\tAccept larger code by factor x\n"
+            "  --config name\t\t\tUse linker config file\n"
+            "  --cpu type\t\t\tSet cpu type\n"
+            "  --create-dep name\t\tCreate a make dependency file\n"
+            "  --create-full-dep name\tCreate a full make dependency file\n"
+            "  --data-label name\t\tDefine and export a DATA segment label\n"
+            "  --data-name seg\t\tSet the name of the DATA segment\n"
+            "  --debug\t\t\tDebug mode\n"
+            "  --debug-info\t\t\tAdd debug info\n"
+            "  --feature name\t\tSet an emulation feature\n"
+            "  --force-import sym\t\tForce an import of symbol `sym'\n"
+            "  --forget-inc-paths\t\tForget include search paths (compiler)\n"
+            "  --help\t\t\tHelp (this text)\n"
+            "  --include-dir dir\t\tSet a compiler include directory path\n"
+            "  --ld-args options\t\tPass options to the linker\n"
+            "  --lib file\t\t\tLink this library\n"
+            "  --lib-path path\t\tSpecify a library search path\n"
+            "  --list-targets\t\tList all available targets\n"
+            "  --listing name\t\tCreate an assembler listing file\n"
+            "  --list-bytes n\t\tNumber of bytes per assembler listing line\n"
+            "  --mapfile name\t\tCreate a map file\n"
+            "  --memory-model model\t\tSet the memory model\n"
+            "  --module\t\t\tLink as a module\n"
+            "  --module-id id\t\tSpecify a module id for the linker\n"
+            "  --o65-model model\t\tOverride the o65 model\n"
+            "  --obj file\t\t\tLink this object file\n"
+            "  --obj-path path\t\tSpecify an object file search path\n"
+            "  --register-space b\t\tSet space available for register variables\n"
+            "  --register-vars\t\tEnable register variables\n"
+            "  --rodata-name seg\t\tSet the name of the RODATA segment\n"
+            "  --signed-chars\t\tDefault characters are signed\n"
+            "  --standard std\t\tLanguage standard (c89, c99, cc65)\n"
+            "  --start-addr addr\t\tSet the default start address\n"
+            "  --static-locals\t\tMake local variables static\n"
+            "  --target sys\t\t\tSet the target system\n"
+            "  --version\t\t\tPrint the version number\n"
+            "  --verbose\t\t\tVerbose mode\n"
+            "  --zeropage-label name\t\tDefine and export a ZEROPAGE segment label\n"
+            "  --zeropage-name seg\t\tSet the name of the ZEROPAGE segment\n",
             ProgName);
 }
 
@@ -726,7 +766,7 @@ static void OptAsmArgs (const char* Opt attribute ((unused)), const char* Arg)
 }
 
 
-
+                                                            
 static void OptAsmDefine (const char* Opt attribute ((unused)), const char* Arg)
 /* Define an assembler symbol (assembler) */
 {
@@ -743,6 +783,14 @@ static void OptAsmIncludeDir (const char* Opt attribute ((unused)), const char*
 
 
 
+static void OptBinIncludeDir (const char* Opt attribute ((unused)), const char* Arg)
+/* Binary include directory (assembler) */
+{
+    CmdAddArg2 (&CA65, "--bin-include-dir", Arg);
+}
+
+
+
 static void OptBssLabel (const char* Opt attribute ((unused)), const char* Arg)
 /* Handle the --bss-label option */
 {
@@ -823,11 +871,26 @@ static void OptCPU (const char* Opt attribute ((unused)), const char* Arg)
 
 
 
-static void OptCreateDep (const char* Opt attribute ((unused)),
-                         const char* Arg attribute ((unused)))
+static void OptCreateDep (const char* Opt attribute ((unused)), const char* Arg)
 /* Handle the --create-dep option */
 {
-    CmdAddArg (&CC65, "--create-dep");
+    /* Add the file name to the compiler */
+    CmdAddArg2 (&CC65, "--create-dep", Arg);
+
+    /* Remember the file name for the assembler */
+    DepName = Arg;
+}
+
+
+
+static void OptCreateFullDep (const char* Opt attribute ((unused)), const char* Arg)
+/* Handle the --create-full-dep option */
+{
+    /* Add the file name to the compiler */
+    CmdAddArg2 (&CC65, "--create-full-dep", Arg);
+
+    /* Remember the file name for the assembler */
+    FullDepName = Arg;
 }
 
 
@@ -946,17 +1009,16 @@ static void OptListBytes (const char* Opt attribute ((unused)), const char* Arg)
 
 
 
-static void OptListing (const char* Opt attribute ((unused)),
-                       const char* Arg attribute ((unused)))
+static void OptListing (const char* Opt attribute ((unused)), const char* Arg)
 /* Create an assembler listing */
 {
-    CmdAddArg (&CA65, "-l");
+    CmdAddArg2 (&CA65, "-l", Arg);
 }
 
 
 
 static void OptListTargets (const char* Opt attribute ((unused)),
-                           const char* Arg attribute ((unused)))
+                           const char* Arg attribute ((unused)))
 /* List all targets */
 {
     unsigned I;
@@ -1128,7 +1190,7 @@ static void OptVersion (const char* Opt attribute ((unused)),
 /* Print version number */
 {
     fprintf (stderr,
-            "cl65 V%s - (C) Copyright 1998-2009 Ullrich von Bassewitz\n",
+            "cl65 V%s - (C) Copyright 1998-2011 Ullrich von Bassewitz\n",
             GetVersionAsString ());
 }
 
@@ -1159,6 +1221,7 @@ int main (int argc, char* argv [])
         { "--asm-args",         1,      OptAsmArgs              },
                { "--asm-define",       1,      OptAsmDefine            },
        { "--asm-include-dir",  1,      OptAsmIncludeDir        },
+        { "--bin-include-dir",  1,      OptBinIncludeDir        },
                { "--bss-label",        1,      OptBssLabel             },
        { "--bss-name",         1,      OptBssName              },
                { "--cfg-path",         1,      OptCfgPath              },
@@ -1168,7 +1231,8 @@ int main (int argc, char* argv [])
        { "--codesize",         1,      OptCodeSize             },
         { "--config",           1,      OptConfig               },
         { "--cpu",                     1,      OptCPU                  },
-       { "--create-dep",       0,      OptCreateDep            },
+       { "--create-dep",       1,      OptCreateDep            },
+       { "--create-full-dep",  1,      OptCreateFullDep        },
                { "--data-label",       1,      OptDataLabel            },
        { "--data-name",        1,      OptDataName             },
        { "--debug",            0,      OptDebug                },
@@ -1182,7 +1246,7 @@ int main (int argc, char* argv [])
                { "--lib",              1,      OptLib                  },
                { "--lib-path",         1,      OptLibPath              },
        { "--list-targets",     0,      OptListTargets          },
-       { "--listing",          0,      OptListing              },
+       { "--listing",          1,      OptListing              },
         { "--list-bytes",       1,      OptListBytes            },
        { "--mapfile",          1,      OptMapFile              },
         { "--memory-model",     1,      OptMemoryModel          },
@@ -1273,7 +1337,8 @@ int main (int argc, char* argv [])
 
                case 'S':
                    /* Dont assemble and link the created files */
-                   DontLink = DontAssemble = 1;
+                    DoAssemble = 0;
+                   DoLink     = 0;
                    break;
 
                case 'T':
@@ -1311,7 +1376,7 @@ int main (int argc, char* argv [])
 
                case 'c':
                    /* Don't link the resulting files */
-                   DontLink = 1;
+                   DoLink = 0;
                    break;
 
                case 'd':
@@ -1337,7 +1402,7 @@ int main (int argc, char* argv [])
 
                case 'l':
                    /* Create an assembler listing */
-                   OptListing (Arg, 0);
+                   OptListing (Arg, GetArg (&I, 2));
                    break;
 
                case 'm':
@@ -1395,7 +1460,7 @@ int main (int argc, char* argv [])
 
                case FILETYPE_ASM:
                    /* Assemble the file */
-                   if (!DontAssemble) {
+                   if (DoAssemble) {
                        Assemble (Arg);
                    }
                    break;
@@ -1433,7 +1498,7 @@ int main (int argc, char* argv [])
     }
 
     /* Link the given files if requested and if we have any */
-    if (DontLink == 0 && LD65.FileCount > 0) {
+    if (DoLink && LD65.FileCount > 0) {
        Link ();
     }