]> git.saurik.com Git - bison.git/commitdiff
location: pass the location first
authorAkim Demaille <akim@lrde.epita.fr>
Fri, 1 Feb 2013 13:24:48 +0000 (14:24 +0100)
committerAkim Demaille <akim@lrde.epita.fr>
Fri, 1 Feb 2013 13:24:48 +0000 (14:24 +0100)
* src/location.h, src/location.c (location_print): For consistency
with other data structures and other location_* routines, pass the
location argument first.
* src/complain.c: Adjust.
(location_caret): Likewise.
* src/parse-gram.y: Adjust.

src/complain.c
src/location.c
src/location.h
src/parse-gram.y

index c6e03d51f9503577aa600719e774dbebeb8f578c..5a1b45a43a4a75a7f93dbb92c26077e3046fc45a 100644 (file)
@@ -89,7 +89,7 @@ error_message (const location *loc, warnings flags, const char *prefix,
   unsigned pos = 0;
 
   if (loc)
-    pos += location_print (stderr, *loc);
+    pos += location_print (*loc, stderr);
   else
     pos += fprintf (stderr, "%s", current_file ? current_file : program_name);
   pos += fprintf (stderr, ": ");
@@ -117,7 +117,7 @@ error_message (const location *loc, warnings flags, const char *prefix,
         putc ('\n', stderr);
         fflush (stderr);
         if (loc && feature_flag & feature_caret && !(flags & no_caret))
-          location_caret (stderr, *loc);
+          location_caret (*loc, stderr);
       }
   }
   fflush (stderr);
index d06f4faea24b00e30ba5095f12f17c90a4ad62ba..ecfa60ee7a49929753572c02e7cb81add03e6f02 100644 (file)
@@ -99,7 +99,7 @@ location_compute (location *loc, boundary *cur, char const *token, size_t size)
 /* Output to OUT the location LOC.
    Warning: it uses quotearg's slot 3.  */
 unsigned
-location_print (FILE *out, location loc)
+location_print (location loc, FILE *out)
 {
   unsigned res = 0;
   int end_col = 0 != loc.end.column ? loc.end.column - 1 : 0;
@@ -161,7 +161,7 @@ cleanup_caret ()
 }
 
 void
-location_caret (FILE *out, location loc)
+location_caret (location loc, FILE *out)
 {
   /* FIXME: find a way to support multifile locations, and only open once each
      file. That would make the procedure future-proof.  */
index f739652425dfeefebaad718539865bab7518524d..2783f912a5165a3d2f20e85357180c815392a2aa 100644 (file)
@@ -100,14 +100,14 @@ void location_compute (location *loc,
 
 /* Print location to file. Return number of actually printed
    characters.  */
-unsigned location_print (FILE *out, location loc);
+unsigned location_print (location loc, FILE *out);
 
 /* Free any allocated ressources and close any open file handles that are
    left-over by the usage of location_caret.  */
 void cleanup_caret (void);
 
 /* Output to OUT the line and caret corresponding to location LOC.  */
-void location_caret (FILE *out, location loc);
+void location_caret (location loc, FILE *out);
 
 /* Return -1, 0, 1, depending whether a is before, equal, or
    after b.  */
index 8a2a04ea32e2e27551caf7c384b299f6f459736d..0b42004104237f36b83196205ab77b29fe296173 100644 (file)
@@ -40,7 +40,7 @@
 static YYLTYPE lloc_default (YYLTYPE const *, int);
 
 #define YY_LOCATION_PRINT(File, Loc) \
-          location_print (File, Loc)
+          location_print (Loc, File)
 
 static void version_check (location const *loc, char const *version);