🚀 go-pugleaf

The following patch applies to libg++-2.3/libg++/etc/trie-gen/compact.cc
and fixes a problem with statements of the form printf("...",i,++i) which
genberate results that are dependent on the order in which the arguments
are evaluated.

This bug also affects at least libg++-2.2.

*** xcompact.cc	Tue Dec  1 00:57:07 1992
--- compact.cc	Wed Dec 23 17:35:22 1992
***************
*** 303,310 ****
            max_number < MAX_UNSIGNED_CHAR ? "char" : max_number < MAX_UNSIGNED_SHORT ? "short" : "int",
            current_rows);
        
!   for (i = 0; i < current_rows; )
!     printf ("%*d,%s", field_width, row_offsets[i], ++i % COL_WIDTH ? " " : "\n  ");
        
    max_number = 0;
  
--- 303,310 ----
            max_number < MAX_UNSIGNED_CHAR ? "char" : max_number < MAX_UNSIGNED_SHORT ? "short" : "int",
            current_rows);
        
!   for (i = 0; i < current_rows; i++ )
!     printf ("%*d,%s", field_width, row_offsets[i], (i+1) % COL_WIDTH ? " " : "\n  ");
        
    max_number = 0;
  
***************
*** 319,326 ****
            max_number < MAX_UNSIGNED_CHAR ? "char" : max_number < MAX_UNSIGNED_SHORT ? "short" : "int",
            compressed_len + 1);
        
!   for (i = 0; i < compressed_len + 1; )
!     printf ("%*d,%s", field_width, checks[i], ++i % COL_WIDTH ? " " : "\n  ");
        
    max_number = 0;
  
--- 319,326 ----
            max_number < MAX_UNSIGNED_CHAR ? "char" : max_number < MAX_UNSIGNED_SHORT ? "short" : "int",
            compressed_len + 1);
        
!   for (i = 0; i < compressed_len + 1; i++)
!     printf ("%*d,%s", field_width, checks[i], (i+1) % COL_WIDTH ? " " : "\n  ");
        
    max_number = 0;
  
***************
*** 335,342 ****
            max_number < MAX_SIGNED_CHAR ? "char" : max_number < MAX_SIGNED_SHORT ? "short" : "int",
            compressed_len + 1);
        
!   for (i = 0; i <= compressed_len; )
!     printf ("%*d,%s", field_width, values[i], ++i % COL_WIDTH ? " " : "\n  ");
        
    printf ("\n};\n\n");
  }
--- 335,342 ----
            max_number < MAX_SIGNED_CHAR ? "char" : max_number < MAX_SIGNED_SHORT ? "short" : "int",
            compressed_len + 1);
        
!   for (i = 0; i <= compressed_len; i++ )
!     printf ("%*d,%s", field_width, values[i], (i+1) % COL_WIDTH ? " " : "\n  ");
        
    printf ("\n};\n\n");
  }

-- 
Jeff Smith, Computer Science, Warwick University, Coventry, CV4 7AL, England
jeff@dcs.warwick.ac.uk  phone: +44 203 523485   fax: +44 203 525714