]> Shamusworld >> Repos - rmac/blobdiff - error.c
Version bump for last commit. :-)
[rmac] / error.c
diff --git a/error.c b/error.c
index 4f65cc42c50abc9601a051d688b508d51a99553e..47cff9f29b4fae42afc48f96dade5e5eb92d351b 100644 (file)
--- a/error.c
+++ b/error.c
@@ -1,63 +1,73 @@
 //
-// RMAC - Reboot's Macro Assembler for the Atari Jaguar Console System
+// RMAC - Renamed Macro Assembler for all Atari computers
 // ERROR.C - Error Handling
-// Copyright (C) 199x Landon Dyer, 2011 Reboot and Friends
+// Copyright (C) 199x Landon Dyer, 2011-2021 Reboot and Friends
 // RMAC derived from MADMAC v1.07 Written by Landon Dyer, 1986
-// Source Utilised with the Kind Permission of Landon Dyer
+// Source utilised with the kind permission of Landon Dyer
 //
 
 #include "error.h"
+#include <stdarg.h>
 #include "token.h"
 #include "listing.h"
-
+char * interror_msg[] = {
+       "Unknown internal error",       // Error not referenced, should not be displayed
+       "Unknown internal error",       // Error not referenced, should not be displayed
+       "Bad MULTX entry in chrtab",                            // Error #2
+       "Unknown internal error",       // Error not referenced, should not be displayed
+       "Bad fixup type",                                                       // Error #4
+       "Bad operator in expression stream",            // Error #5
+       "Can't find generated code in section",         // Error #6
+       "Fixup (loc) out of range",                                     // Error #7
+       "Absolute top filename found",                          // Error #8
+       "The RISC expression evaluator blew up, sorry" // Error #9
+};
+
+// Exported variables
 int errcnt;                                            // Error count
 char * err_fname;                              // Name of error message file
 
-static const char nl[] = "\n";
+// Internal variables
 static long unused;                            // For supressing 'write' warnings
 
-
 //
 // Report error if not at EOL
 //
-int at_eol(void)
+// N.B.: Since this should *never* happen, we can feel free to add whatever
+//       diagnostics that will help in tracking down a problem to this function.
+//
+int ErrorIfNotAtEOL(void)
 {
-       char msg[256];
-
        if (*tok != EOL)
        {
-//             error("syntax error");
-               sprintf(msg, "syntax error. expected EOL, found $%X ('%c')", *tok, *tok);
-               error(msg);
+               error("syntax error. expected EOL, found $%X ('%c')", *tok, *tok);
+               printf("Token = ");
+               DumpToken(*tok);
+               printf("\n");
+               DumpTokenBuffer();
        }
 
        return 0;
 }
 
-
 //
-// Cannot Create a File
+// Cannot create a file
 //
-void cantcreat(const char * fn)
+void CantCreateFile(const char * fn)
 {
-       printf("cannot create: '%s'\n", fn);
+       printf("Cannot create file: '%s'\n", fn);
        exit(1);
 }
 
-
 //
-// Setup for Error Message
-// o  Create error listing file (if necessary)
-// o  Set current filename
+// Setup for error message
+//  o  Create error listing file (if necessary)
+//  o  Set current filename
 //
 void err_setup(void)
 {
        char fnbuf[FNSIZ];
 
-// This seems like it's unnecessary, as token.c seems to take care of this all by itself.
-// Can restore if it's really needed. If not, into the bit bucket it goes. :-)
-//     setfnum(cfileno);
-
        if (err_fname != NULL)
        {
                strcpy(fnbuf, err_fname);
@@ -68,53 +78,78 @@ void err_setup(void)
                err_fname = NULL;
 
                if ((err_fd = open(fnbuf, _OPEN_FLAGS, _PERM_MODE)) < 0)
-                       cantcreat(fnbuf);
+                       CantCreateFile(fnbuf);
 
                err_flag = 1;
        }
 }
 
-
 //
-// Display Error Message
+// Display error message (uses printf() style variable arguments)
 //
-int error(const char * s)
-{
-       char buf[EBUFSIZ];
-       unsigned int length;
-
-       err_setup();
-
-       if (listing > 0)
-               ship_ln(s);
-
-       sprintf(buf, "%s %d: Error: %s%s", curfname, curlineno, s, nl);
-       length = strlen(buf);
-
-       if (err_flag)
-               unused = write(err_fd, buf, length);
-       else
-               printf("%s", buf);
-
-       taglist('E');
-       errcnt++;
-
-       return ERROR;
-}
-
-
-int errors(const char * s, char * s1)
+int error(const char * text, ...)
 {
        char buf[EBUFSIZ];
        char buf1[EBUFSIZ];
 
        err_setup();
-       sprintf(buf, s, s1);
+
+       va_list arg;
+       va_start(arg, text);
+       vsprintf(buf, text, arg);
+       va_end(arg);
 
        if (listing > 0)
                ship_ln(buf);
 
-       sprintf(buf1, "%s %d: Error: %s%s", curfname, curlineno, buf, nl);
+       if (cur_inobj)
+       {
+               switch (cur_inobj->in_type)
+               {
+               case SRC_IFILE:
+                       sprintf(buf1, "%s %d: Error: %s\n", curfname, curlineno, buf);
+                       break;
+               case SRC_IMACRO:
+               {
+                       // This is basically SetFilenameForErrorReporting() but we don't
+                       // call it here as it will clobber curfname. That function is used
+                       // during fixups only so it really doesn't matter at that point...
+                       char * filename;
+                       FILEREC * fr;
+                       uint16_t fnum = cur_inobj->inobj.imacro->im_macro->cfileno;
+
+                       // Check for absolute top filename (this should never happen)
+                       if (fnum == -1)
+                               interror(8);
+                       else
+                       {
+                               fr = filerec;
+
+                               // Advance to the correct record...
+                               while (fr != NULL && fnum != 0)
+                               {
+                                       fr = fr->frec_next;
+                                       fnum--;
+                               }
+                       }
+                       // Check for file # record not found (this should never happen either)
+                       if (fr == NULL)
+                               interror(8);
+
+                       filename = fr->frec_name;
+
+                       sprintf(buf1, "%s %d: Error: %s\nCalled from: %s %d\n", filename, cur_inobj->inobj.imacro->im_macro->lineList->lineno, buf,
+                       curfname, curlineno);
+               }
+                       break;
+               case SRC_IREPT:
+                       sprintf(buf1, "%s %d: Error: %s\n", curfname, cur_inobj->inobj.irept->lineno, buf);
+                       break;
+               }
+       }
+       else
+               // No current file so cur_inobj is NULL
+               sprintf(buf1, "%s %d: Error: %s\n", curfname, curlineno, buf);
 
        if (err_flag)
                unused = write(err_fd, buf1, (LONG)strlen(buf1));
@@ -122,70 +157,29 @@ int errors(const char * s, char * s1)
                printf("%s", buf1);
 
        taglist('E');
-       ++errcnt;
+       errcnt++;
 
        return ERROR;
 }
 
-
-int warn(const char * s)
-{
-       char buf[EBUFSIZ];
-
-       err_setup();
-
-       if (listing > 0)
-               ship_ln(s);
-
-       sprintf(buf, "%s %d: Warning: %s%s", curfname, curlineno, s, nl);
-
-       if (err_flag)
-               unused = write(err_fd, buf, (LONG)strlen(buf));
-       else
-               printf("%s", buf);
-
-       taglist('W');
-
-       return OK;
-}
-
-
-int warns(const char * s, char * s1)
-{
-       char buf[EBUFSIZ];
-       char buf1[EBUFSIZ];
-
-       err_setup();
-       sprintf(buf, s, s1);
-
-       if (listing > 0)
-               ship_ln(s);
-
-       sprintf(buf1, "%s %d: Warning: %s%s", curfname, curlineno, buf, nl);
-
-       if (err_flag)
-               unused = write(err_fd, buf1, (LONG)strlen(buf1));
-       else
-               printf("%s", buf1);
-
-       taglist('W');
-
-       return OK;
-}
-
-
-int warni(const char * s, unsigned i)
+//
+// Display warning message (uses printf() style variable arguments)
+//
+int warn(const char * text, ...)
 {
        char buf[EBUFSIZ];
        char buf1[EBUFSIZ];
 
        err_setup();
-       sprintf(buf, s, i);
+       va_list arg;
+       va_start(arg, text);
+       vsprintf(buf, text, arg);
+       va_end(arg);
 
        if (listing > 0)
                ship_ln(buf);
 
-       sprintf(buf1, "%s %d: Warning: %s%s", curfname, curlineno, buf, nl);
+       sprintf(buf1, "%s %d: Warning: %s\n", curfname, curlineno, buf);
 
        if (err_flag)
                unused = write(err_fd, buf1, (LONG)strlen(buf1));
@@ -197,7 +191,6 @@ int warni(const char * s, unsigned i)
        return OK;
 }
 
-
 int fatal(const char * s)
 {
        char buf[EBUFSIZ];
@@ -207,7 +200,7 @@ int fatal(const char * s)
        if (listing > 0)
                ship_ln(s);
 
-       sprintf(buf, "%s %d: Fatal: %s%s", curfname, curlineno, s, nl);
+       sprintf(buf, "%s %d: Fatal: %s\n", curfname, curlineno, s);
 
        if (err_flag)
                unused = write(err_fd, buf, (LONG)strlen(buf));
@@ -217,13 +210,12 @@ int fatal(const char * s)
        exit(1);
 }
 
-
 int interror(int n)
 {
        char buf[EBUFSIZ];
 
        err_setup();
-       sprintf(buf, "%s %d: Internal Error Number %d%s", curfname, curlineno, n, nl);
+       sprintf(buf, "%s %d: Internal error #%d: %s\n", curfname, curlineno, n, interror_msg[n]);
 
        if (listing > 0)
                ship_ln(buf);
@@ -235,4 +227,3 @@ int interror(int n)
 
        exit(1);
 }
-