ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/radiance/ray/src/util/radcompare.c
(Generate patch)

Comparing ray/src/util/radcompare.c (file contents):
Revision 2.1 by greg, Mon Oct 15 17:52:52 2018 UTC vs.
Revision 2.14 by greg, Fri Oct 19 23:56:15 2018 UTC

# Line 33 | Line 33 | double max_lim = 0.25;         /* difference limit if non-neg
33  
34   int     lin1cnt=0, lin2cnt=0;   /* file line position */
35  
36 + const char      nsuffix[10][3] = {              /* 1st, 2nd, 3rd, etc. */
37 +                        "th","st","nd","rd","th","th","th","th","th","th"
38 +                };
39 + #define num_sfx(n)      nsuffix[(n)%10]
40 +
41                                  /* file types */
42   const char      *file_type[] = {
43                          "Unrecognized",
# Line 40 | Line 45 | const char     *file_type[] = {
45                          "ascii",
46                          COLRFMT,
47                          CIEFMT,
48 <                        "BINARY_float",
49 <                        "BINARY_double",
48 >                        "float",
49 >                        "double",
50                          "BSDF_RBFmesh",
51                          "Radiance_octree",
52                          "Radiance_tmesh",
53                          "BINARY_unknown",
54                          NULL    /* terminator */
55                  };
56 <
56 >                                /* keep consistent with above */
57   enum {TYP_UNKNOWN, TYP_TEXT, TYP_ASCII, TYP_RGBE, TYP_XYZE, TYP_FLOAT,
58                  TYP_DOUBLE, TYP_RBFMESH, TYP_OCTREE, TYP_TMESH, TYP_BINARY};
59                  
# Line 69 | Line 74 | LUTAB  hdr2 = LU_SINIT(free,free);
74   #define adv_linecnt(htp)        (lin1cnt += (htp == &hdr1), \
75                                          lin2cnt += (htp == &hdr2))
76  
77 + typedef struct {                /* dynamic line buffer */
78 +        char    *str;
79 +        int     len;
80 +        int     siz;
81 + } LINEBUF;
82 +
83 + #define init_line(bp)   ((bp)->str = NULL, (bp)->siz = 0)
84 +                                /* 100 MByte limit on line buffer */
85 + #define MAXBUF          (100L<<20)
86 +
87                                  /* input files */
88   char            *progname = NULL;
89   const char      stdin_name[] = "<stdin>";
90   const char      *f1name=NULL, *f2name=NULL;
91   FILE            *f1in=NULL, *f2in=NULL;
92 +
93                                  /* running real differences */
94   double  diff2sum = 0;
95 < int     nsum = 0;
95 > long    nsum = 0;
96  
97   /* Report usage and exit */
98   static void
# Line 84 | Line 100 | usage()
100   {
101          fputs("Usage: ", stderr);
102          fputs(progname, stderr);
103 <        fputs(" [-h][-s|-w|-v][-rel min_test][-rms epsilon][-max epsilon] file1 file2\n",
103 >        fputs(" [-h][-s|-w|-v][-rel min_test][-rms epsilon][-max epsilon] reference test\n",
104                          stderr);
105 <        exit(1);
105 >        exit(2);
106   }
107  
108 + /* Read a text line, increasing buffer size as necessary */
109 + static int
110 + read_line(LINEBUF *bp, FILE *fp)
111 + {
112 +        bp->len = 0;
113 +        if (!bp->str) {
114 +                bp->str = (char *)malloc(bp->siz = 512);
115 +                if (!bp->str)
116 +                        goto memerr;
117 +        }
118 +        while (fgets(bp->str + bp->len, bp->siz - bp->len, fp)) {
119 +                bp->len += strlen(bp->str + bp->len);
120 +                if (bp->str[bp->len-1] == '\n')
121 +                        break;          /* found EOL */
122 +                if (bp->len < bp->siz - 4)
123 +                        continue;       /* at EOF? */
124 +                if (bp->siz >= MAXBUF)
125 +                        break;          /* don't go to extremes */
126 +                if ((bp->siz += bp->siz/2) > MAXBUF)
127 +                        bp->siz = MAXBUF;
128 +                bp->str = (char *)realloc(bp->str, bp->siz);
129 +                if (!bp->str)
130 +                        goto memerr;
131 +        }
132 +        return(bp->len);
133 + memerr:
134 +        fprintf(stderr,
135 +                "%s: out of memory in read_line() allocating %d byte buffer\n",
136 +                        progname, bp->siz);
137 +        exit(2);
138 + }
139 +
140 + /* Free line buffer */
141 + static void
142 + free_line(LINEBUF *bp)
143 + {
144 +        if (bp->str) free(bp->str);
145 +        init_line(bp);
146 + }
147 +
148   /* Get type ID from name (or 0 if not found) */
149   static int
150   xlate_type(const char *nm)
# Line 116 | Line 172 | real_check(double r1, double r2)
172          }
173          if (max_lim >= 0 && diff2 > max_lim*max_lim) {
174                  if (report != REP_QUIET)
175 <                        fprintf(stderr,
176 <                        "%s: %sdifference between %.8g and %.8g exceeds epsilon\n",
175 >                        printf(
176 >                        "%s: %sdifference between %.8g and %.8g exceeds epsilon of %.8g\n",
177                                          progname,
178                                          (rel_min > 0) ? "relative " : "",
179 <                                        r1, r2);
179 >                                        r1, r2, max_lim);
180                  return(0);
181          }
182          diff2sum += diff2;
# Line 128 | Line 184 | real_check(double r1, double r2)
184          return(1);
185   }
186  
187 + /* Compare two color values for equivalence */
188 + static int
189 + color_check(COLOR c1, COLOR c2)
190 + {
191 +        int     p;
192 +
193 +        if (!real_check(colval(c1,RED)+colval(c1,GRN)+colval(c1,BLU)*(1./3.),
194 +                        colval(c2,RED)+colval(c2,GRN)+colval(c2,BLU))*(1./3.))
195 +                return(0);
196 +
197 +        p = (colval(c1,GRN) > colval(c1,RED)) ? GRN : RED;
198 +        if (colval(c1,BLU) > colval(c1,p)) p = BLU;
199 +        
200 +        return(real_check(colval(c1,p), colval(c2,p)));
201 + }
202 +
203   /* Compare two strings for equivalence */
204   static int
205   equiv_string(char *s1, char *s2)
# Line 138 | Line 210 | equiv_string(char *s1, char *s2)
210                                  /* skip whitespace at beginning */
211          while (isspace(*s1)) s1++;
212          while (isspace(*s2)) s2++;
141 –        if (!strcmp(s1, s2))    /* quick check */
142 –                return(1);
213          while (*s1) {           /* check each word */
214 <                int     inquote = *s1;
214 >                int     inquote;
215 >                if (!*s2)       /* unexpected EOL in s2? */
216 >                        return(0);
217 >                inquote = *s1;
218                  if ((inquote != '\'') & (inquote != '"'))
219                          inquote = 0;
220                  if (inquote) {  /* quoted text must match exactly */
# Line 205 | Line 278 | setheadvar(char *val, void *p)
278          int     kln, vln;
279          int     n;
280  
281 +        adv_linecnt(htp);       /* side-effect is to count lines */
282          if (!isalpha(*val))     /* key must start line */
283                  return(0);
284          key = val++;
# Line 220 | Line 294 | setheadvar(char *val, void *p)
294                  val++;
295          if (!*val)              /* nothing set? */
296                  return(0);
223 –        vln = strlen(val);      /* eliminate space and newline at end */
224 –        while (--vln > 0 && isspace(val[vln]))
225 –                ;
226 –        val[++vln] = '\0';
297                                  /* check if key to ignore */
298          for (n = 0; hdr_ignkey[n]; n++)
299                  if (!strcmp(key, hdr_ignkey[n]))
300                          return(0);
301 +        vln = strlen(val);      /* eliminate space and newline at end */
302 +        while (isspace(val[--vln]))
303 +                ;
304 +        val[++vln] = '\0';
305          if (!(tep = lu_find(htp, key)))
306 <                return(-1);
306 >                return(-1);     /* memory allocation error */
307          if (!tep->key)
308                  tep->key = strcpy(malloc(kln+1), key);
309          if (tep->data)
310                  free(tep->data);
311          tep->data = strcpy(malloc(vln+1), val);
238 –        adv_linecnt(htp);
312          return(1);
313   }
314  
# Line 246 | Line 319 | match_val(const LUENT *ep1, void *p2)
319          const LUENT     *ep2 = lu_find((LUTAB *)p2, ep1->key);
320          if (!ep2 || !ep2->data) {
321                  if (report != REP_QUIET)
322 <                        fprintf(stderr, "%s: Variable '%s' missing in '%s'\n",
322 >                        printf("%s: variable '%s' missing in '%s'\n",
323                                          progname, ep1->key, f2name);
324                  return(-1);
325          }
326          if (!equiv_string((char *)ep1->data, (char *)ep2->data)) {
327 <                if (report != REP_QUIET)
328 <                        fprintf(stderr, "%s: Header variables '%s' have different values\n",
327 >                if (report != REP_QUIET) {
328 >                        printf("%s: header variable '%s' has different values\n",
329                                          progname, ep1->key);
330 +                        if (report >= REP_VERBOSE) {
331 +                                printf("%s: %s=%s\n", f1name,
332 +                                                ep1->key, (char *)ep1->data);
333 +                                printf("%s: %s=%s\n", f2name,
334 +                                                ep2->key, (char *)ep2->data);
335 +                        }
336 +                }
337                  return(-1);
338          }
339          return(1);              /* good match */
# Line 261 | Line 341 | match_val(const LUENT *ep1, void *p2)
341  
342   /* Compare two sets of header variables */
343   static int
344 < headers_match(LUTAB *hp1, LUTAB *hp2)
344 > headers_match()
345   {
346 <        int     ne = lu_doall(hp1, match_val, hp2);
346 >        int     ne = lu_doall(&hdr1, match_val, &hdr2);
347          if (ne < 0)
348                  return(0);      /* something didn't match! */
349 <        ne = lu_doall(hp2, NULL, NULL) - ne;
350 <        if (ne) {
351 <                if (report != REP_QUIET)
272 <                        fprintf(stderr, "%s: '%s' has %d extra header variable(s)\n",
349 >                                /* non-fatal if second header has extra */
350 >        if (report >= REP_WARN && (ne = lu_doall(&hdr2, NULL, NULL) - ne))
351 >                printf("%s: warning - '%s' has %d extra header setting(s)\n",
352                                          progname, f2name, ne);
274 –                return(0);
275 –        }
353          return(1);              /* good match */
354   }
355  
# Line 284 | Line 361 | input_is_binary(FILE *fin)
361          int     c = 0;
362  
363          while ((c = getc(fin)) != EOF) {
364 +                ++n;
365                  if (!c | (c > 127))
366                          break;                  /* non-ascii character */
367 <                if (++n >= 10240)
367 >                if (n >= 10240)
368                          break;                  /* enough to be confident */
369          }
370          if (!n)
# Line 316 | Line 394 | identify_type(const char *name, FILE *fin, LUTAB *htp)
394                  char    sbuf[32];
395                  if (!fgets(sbuf, sizeof(sbuf), fin))
396                          goto badeof;
397 +                adv_linecnt(htp);               /* for #?ID string */
398                  if (report >= REP_WARN && strncmp(sbuf, "RADIANCE", 8)) {
399 <                        fputs(name, stderr);
400 <                        fputs(": warning - unexpected header ID: ", stderr);
401 <                        fputs(sbuf, stderr);
399 >                        fputs(name, stdout);
400 >                        fputs(": warning - unexpected header ID: ", stdout);
401 >                        fputs(sbuf, stdout);
402                  }
324 –                adv_linecnt(htp);               /* for #?ID string */
403                  if (getheader(fin, setheadvar, htp) < 0) {
404                          fputs(name, stderr);
405 <                        fputs(": Unknown error reading header\n", stderr);
405 >                        fputs(": unknown error reading header\n", stderr);
406                          return(-1);
407                  }
408                  adv_linecnt(htp);               /* for trailing emtpy line */
# Line 342 | Line 420 | identify_type(const char *name, FILE *fin, LUTAB *htp)
420          return(TYP_TEXT);
421   badeof:
422          if (report != REP_QUIET) {
423 <                fputs(name, stderr);
424 <                fputs(": Unexpected end-of-file\n", stderr);
423 >                fputs(name, stdout);
424 >                fputs(": unexpected end-of-file\n", stdout);
425          }
426          return(-1);
427   }
# Line 356 | Line 434 | good_RMS()
434                  return(1);
435          if (diff2sum/(double)nsum > rms_lim*rms_lim) {
436                  if (report != REP_QUIET)
437 <                        fprintf(stderr,
438 <        "%s: %sRMS difference between '%s' and '%s' of %.5g exceeds limit\n",
437 >                        printf(
438 >        "%s: %sRMS difference between '%s' and '%s' of %.5g exceeds limit of %.5g\n",
439                                          progname,
440                                          (rel_min > 0) ? "relative " : "",
441                                          f1name, f2name,
442 <                                        sqrt(diff2sum/(double)nsum));
442 >                                        sqrt(diff2sum/(double)nsum), rms_lim);
443                  return(0);
444          }
445          if (report >= REP_VERBOSE)
446 <                fprintf(stderr,
369 <                        "%s: %sRMS difference of reals in '%s' and '%s' is %.5g\n",
446 >                printf("%s: %sRMS difference of reals in '%s' and '%s' is %.5g\n",
447                                  progname, (rel_min > 0) ? "relative " : "",
448                                  f1name, f2name, sqrt(diff2sum/(double)nsum));
449          return(1);
# Line 376 | Line 453 | good_RMS()
453   static int
454   compare_binary()
455   {
456 +        int     c1=0, c2=0;
457 +
458          if (report >= REP_VERBOSE) {
459 <                fputs(progname, stderr);
460 <                fputs(": comparing inputs as binary\n", stderr);
459 >                fputs(progname, stdout);
460 >                fputs(": comparing inputs as binary\n", stdout);
461          }
462          for ( ; ; ) {                           /* exact byte matching */
463 <                int     c1 = getc(f1in);
464 <                int     c2 = getc(f2in);
463 >                c1 = getc(f1in);
464 >                c2 = getc(f2in);
465                  if (c1 == EOF) {
466                          if (c2 == EOF)
467                                  return(1);      /* success! */
468                          if (report != REP_QUIET) {
469 <                                fputs(f1name, stderr);
470 <                                fputs(": Unexpected end-of-file\n", stderr);
469 >                                fputs(f1name, stdout);
470 >                                fputs(": unexpected end-of-file\n", stdout);
471                          }
472                          return(0);
473                  }
474                  if (c2 == EOF) {
475                          if (report != REP_QUIET) {
476 <                                fputs(f2name, stderr);
477 <                                fputs(": Unexpected end-of-file\n", stderr);
476 >                                fputs(f2name, stdout);
477 >                                fputs(": unexpected end-of-file\n", stdout);
478                          }
479                          return(0);
480                  }
# Line 404 | Line 483 | compare_binary()
483          }
484          if (report == REP_QUIET)
485                  return(0);
486 <        fprintf(stderr, "%s: binary files '%s' and '%s' differ at offset %ld|%ld\n",
486 >        printf("%s: binary files '%s' and '%s' differ at byte offset %ld|%ld\n",
487                          progname, f1name, f2name, ftell(f1in), ftell(f2in));
488 +        if (report >= REP_VERBOSE)
489 +                printf("%s: byte in '%s' is 0x%X, byte in '%s' is 0x%X\n",
490 +                                progname, f1name, c1, f2name, c2);
491          return(0);
492   }
493  
# Line 413 | Line 495 | compare_binary()
495   static int
496   compare_text()
497   {
498 <        char    l1buf[4096], l2buf[4096];
498 >        LINEBUF l1buf, l2buf;
499  
500          if (report >= REP_VERBOSE) {
501 <                fputs(progname, stderr);
502 <                fputs(": comparing inputs as ASCII text\n", stderr);
501 >                fputs(progname, stdout);
502 >                fputs(": comparing inputs as ASCII text\n", stdout);
503          }
504 <                                                /* compare a line at a time */
505 <        while (fgets(l1buf, sizeof(l1buf), f1in)) {
504 >        init_line(&l1buf); init_line(&l2buf);   /* compare a line at a time */
505 >        while (read_line(&l1buf, f1in)) {
506                  lin1cnt++;
507 <                if (!*sskip2(l1buf,0))
507 >                if (!*sskip2(l1buf.str,0))
508                          continue;               /* ignore empty lines */
509 <                while (fgets(l2buf, sizeof(l2buf), f2in)) {
509 >
510 >                while (read_line(&l2buf, f2in)) {
511                          lin2cnt++;
512 <                        if (*sskip2(l2buf,0))
512 >                        if (*sskip2(l2buf.str,0))
513                                  break;          /* found other non-empty line */
514                  }
515 <                if (feof(f2in)) {
515 >                if (!l2buf.len) {               /* input 2 EOF? */
516                          if (report != REP_QUIET) {
517 <                                fputs(f2name, stderr);
518 <                                fputs(": Unexpected end-of-file\n", stderr);
517 >                                fputs(f2name, stdout);
518 >                                fputs(": unexpected end-of-file\n", stdout);
519                          }
520 +                        free_line(&l1buf); free_line(&l2buf);
521                          return(0);
522                  }
523                                                  /* compare non-empty lines */
524 <                if (!equiv_string(l1buf, l2buf)) {
524 >                if (!equiv_string(l1buf.str, l2buf.str)) {
525                          if (report != REP_QUIET) {
526 <                                fprintf(stderr, "%s: inputs '%s' and '%s' differ at line %d|%d\n",
526 >                                printf("%s: inputs '%s' and '%s' differ at line %d|%d\n",
527                                                  progname, f1name, f2name,
528                                                  lin1cnt, lin2cnt);
529 <                                if (report >= REP_VERBOSE) {
530 <                                        fputs("------------- Mismatch -------------\n", stderr);
531 <                                        fprintf(stderr, "%s(%d):\t%s", f1name,
532 <                                                        lin1cnt, l1buf);
533 <                                        fprintf(stderr, "%s(%d):\t%s", f2name,
534 <                                                        lin2cnt, l2buf);
529 >                                if ( report >= REP_VERBOSE &&
530 >                                                (l1buf.len < 256) &
531 >                                                (l2buf.len < 256) ) {
532 >                                        fputs("------------- Mismatch -------------\n", stdout);
533 >                                        printf("%s@%d:\t%s", f1name,
534 >                                                        lin1cnt, l1buf.str);
535 >                                        printf("%s@%d:\t%s", f2name,
536 >                                                        lin2cnt, l2buf.str);
537                                  }
538                          }
539 +                        free_line(&l1buf); free_line(&l2buf);
540                          return(0);
541                  }
542          }
543 <                                                /* check for EOF on input 2 */
544 <        while (fgets(l2buf, sizeof(l2buf), f2in)) {
545 <                if (!*sskip2(l2buf,0))
543 >        free_line(&l1buf);                      /* check for EOF on input 2 */
544 >        while (read_line(&l2buf, f2in)) {
545 >                if (!*sskip2(l2buf.str,0))
546                          continue;
547                  if (report != REP_QUIET) {
548 <                        fputs(f1name, stderr);
549 <                        fputs(": Unexpected end-of-file\n", stderr);
548 >                        fputs(f1name, stdout);
549 >                        fputs(": unexpected end-of-file\n", stdout);
550                  }
551 +                free_line(&l2buf);
552                  return(0);
553          }
554 +        free_line(&l2buf);
555          return(good_RMS());                     /* final check for reals */
556   }
557  
# Line 474 | Line 563 | compare_hdr()
563          COLOR   *scan1, *scan2;
564          int     x, y;
565  
566 +        if (report >= REP_VERBOSE) {
567 +                fputs(progname, stdout);
568 +                fputs(": comparing inputs as HDR images\n", stdout);
569 +        }
570          fgetsresolu(&rs1, f1in);
571          fgetsresolu(&rs2, f2in);
572          if (rs1.rt != rs2.rt) {
573                  if (report != REP_QUIET)
574 <                        fprintf(stderr,
574 >                        printf(
575                          "%s: Images '%s' and '%s' have different pixel ordering\n",
576                                          progname, f1name, f2name);
577                  return(0);
578          }
579          if ((rs1.xr != rs2.xr) | (rs1.yr != rs2.yr)) {
580                  if (report != REP_QUIET)
581 <                        fprintf(stderr,
581 >                        printf(
582                          "%s: Images '%s' and '%s' are different sizes\n",
583                                          progname, f1name, f2name);
584                  return(0);
# Line 500 | Line 593 | compare_hdr()
593                  if ((freadscan(scan1, scanlen(&rs1), f1in) < 0) |
594                                  (freadscan(scan2, scanlen(&rs2), f2in) < 0)) {
595                          if (report != REP_QUIET)
596 <                                fprintf(stderr, "%s: Unexpected end-of-file\n",
596 >                                printf("%s: unexpected end-of-file\n",
597                                                  progname);
598                          free(scan1);
599                          free(scan2);
600                          return(0);
601                  }
602 <                for (x = scanlen(&rs1); x--; ) {
603 <                        if (real_check(colval(scan1[x],RED),
511 <                                                colval(scan2[x],RED)) &
512 <                                        real_check(colval(scan1[x],GRN),
513 <                                                colval(scan2[x],GRN)) &
514 <                                        real_check(colval(scan1[x],BLU),
515 <                                                colval(scan2[x],BLU)))
602 >                for (x = 0; x < scanlen(&rs1); x++) {
603 >                        if (color_check(scan1[x], scan2[x]))
604                                  continue;
605                          if (report != REP_QUIET) {
606 <                                fprintf(stderr,
606 >                                printf(
607                                  "%s: pixels at scanline %d offset %d differ\n",
608                                          progname, y, x);
609                                  if (report >= REP_VERBOSE) {
610 <                                        fprintf(stderr, "%s: (R,G,B)=(%g,%g,%g)\n",
610 >                                        printf("%s: (R,G,B)=(%g,%g,%g)\n",
611                                                  f1name, colval(scan1[x],RED),
612                                                  colval(scan1[x],GRN),
613                                                  colval(scan1[x],BLU));
614 <                                        fprintf(stderr, "%s: (R,G,B)=(%g,%g,%g)\n",
614 >                                        printf("%s: (R,G,B)=(%g,%g,%g)\n",
615                                                  f2name, colval(scan2[x],RED),
616                                                  colval(scan2[x],GRN),
617                                                  colval(scan2[x],BLU));
# Line 539 | Line 627 | compare_hdr()
627          return(good_RMS());                     /* final check of RMS */
628   }
629  
542 – const char      nsuffix[10][3] = {              /* 1st, 2nd, 3rd, etc. */
543 –                        "th","st","nd","rd","th","th","th","th","th","th"
544 –                };
545 –
630   /* Compare two inputs that are known to be 32-bit floating-point data */
631   static int
632   compare_float()
# Line 550 | Line 634 | compare_float()
634          long    nread = 0;
635          float   f1, f2;
636  
637 +        if (report >= REP_VERBOSE) {
638 +                fputs(progname, stdout);
639 +                fputs(": comparing inputs as 32-bit IEEE floats\n", stdout);
640 +        }
641          while (getbinary(&f1, sizeof(f1), 1, f1in)) {
642                  if (!getbinary(&f2, sizeof(f2), 1, f2in))
643                          goto badeof;
# Line 557 | Line 645 | compare_float()
645                  if (real_check(f1, f2))
646                          continue;
647                  if (report != REP_QUIET)
648 <                        fprintf(stderr, "%s: %ld%s float values differ\n",
649 <                                        progname, nread, nsuffix[nread%10]);
648 >                        printf("%s: %ld%s float values differ\n",
649 >                                        progname, nread, num_sfx(nread));
650                  return(0);
651          }
652          if (!getbinary(&f2, sizeof(f2), 1, f2in))
653                  return(good_RMS());             /* final check of RMS */
654   badeof:
655          if (report != REP_QUIET)
656 <                fprintf(stderr, "%s: Unexpected end-of-file\n", progname);
656 >                printf("%s: unexpected end-of-file\n", progname);
657          return(0);
658   }
659  
# Line 576 | Line 664 | compare_double()
664          long    nread = 0;
665          double  f1, f2;
666  
667 +        if (report >= REP_VERBOSE) {
668 +                fputs(progname, stdout);
669 +                fputs(": comparing inputs as 64-bit IEEE doubles\n", stdout);
670 +        }
671          while (getbinary(&f1, sizeof(f1), 1, f1in)) {
672                  if (!getbinary(&f2, sizeof(f2), 1, f2in))
673                          goto badeof;
# Line 583 | Line 675 | compare_double()
675                  if (real_check(f1, f2))
676                          continue;
677                  if (report != REP_QUIET)
678 <                        fprintf(stderr, "%s: %ld%s float values differ\n",
679 <                                        progname, nread, nsuffix[nread%10]);
678 >                        printf("%s: %ld%s double values differ\n",
679 >                                        progname, nread, num_sfx(nread));
680                  return(0);
681          }
682          if (!getbinary(&f2, sizeof(f2), 1, f2in))
683                  return(good_RMS());             /* final check of RMS */
684   badeof:
685          if (report != REP_QUIET)
686 <                fprintf(stderr, "%s: Unexpected end-of-file\n", progname);
686 >                printf("%s: unexpected end-of-file\n", progname);
687          return(0);
688   }
689  
# Line 638 | Line 730 | main(int argc, char *argv[])
730                  }
731                  break;
732          }
733 <        if (a != argc-2)
733 >        if (a != argc-2)                        /* make sure of two inputs */
734                  usage();
735 <        if (!f1name) f1name = argv[a];
644 <        if (!f2name) f2name = argv[a+1];
645 <
646 <        if (!strcmp(f1name, f2name)) {          /* inputs are same? */
735 >        if (!strcmp(argv[a], argv[a+1])) {      /* inputs are same? */
736                  if (report >= REP_WARN)
737 <                        fprintf(stderr, "%s: warning - identical inputs given\n",
737 >                        printf("%s: warning - identical inputs given\n",
738                                          progname);
739                  return(0);
740          }
741 +        if (!f1name) f1name = argv[a];
742 +        if (!f2name) f2name = argv[a+1];
743                                                  /* open inputs */
744          SET_FILE_BINARY(stdin);                 /* in case we're using it */
745          if (!f1in && !(f1in = fopen(f1name, "rb"))) {
746                  fprintf(stderr, "%s: cannot open for reading\n", f1name);
747 <                return(1);
747 >                return(2);
748          }
749          if (!strcmp(f2name, "-")) {
750                  f2in = stdin;
751                  f2name = stdin_name;
752          } else if (!(f2in = fopen(f2name, "rb"))) {
753                  fprintf(stderr, "%s: cannot open for reading\n", f2name);
754 <                return(1);
754 >                return(2);
755          }
756                                                  /* load headers */
757          if ((typ1 = identify_type(f1name, f1in, &hdr1)) < 0)
758 <                return(1);
758 >                return(2);
759          if ((typ2 = identify_type(f2name, f2in, &hdr2)) < 0)
760 <                return(1);
760 >                return(2);
761          if (typ1 != typ2) {
762                  if (report != REP_QUIET)
763 <                        fprintf(stderr, "%s: '%s' is %s and '%s' is %s\n",
763 >                        printf("%s: '%s' is %s and '%s' is %s\n",
764                                          progname, f1name, file_type[typ1],
765                                          f2name, file_type[typ2]);
766                  return(1);
767          }
768          ign_header |= !has_header(typ1);        /* check headers if indicated */
769 <        if (!ign_header && !headers_match(&hdr1, &hdr2))
769 >        if (!ign_header && !headers_match())
770                  return(1);
771          lu_done(&hdr1); lu_done(&hdr2);
772          if (!ign_header & (report >= REP_WARN)) {
773                  if (typ1 == TYP_UNKNOWN)
774 <                        fprintf(stderr, "%s: warning - unrecognized format, comparing as binary\n",
774 >                        printf("%s: warning - unrecognized format, comparing as binary\n",
775                                          progname);
776                  if (lin1cnt != lin2cnt)
777 <                        fprintf(stderr, "%s: warning - headers are different lengths\n",
777 >                        printf("%s: warning - headers are different lengths\n",
778                                          progname);
779          }
780          if (report >= REP_VERBOSE)
781 <                fprintf(stderr, "%s: input file type is %s\n",
781 >                printf("%s: input file type is %s\n",
782                                  progname, file_type[typ1]);
783  
784          switch (typ1) {                         /* compare based on type */

Diff Legend

– Removed lines
+ Added lines
< Changed lines
> Changed lines