Have mkerr.pl treat already existing multiline string defs properly
[openssl.git] / util / mkerr.pl
index ba2fd6eba307d86b58a351bd5bfdd01ab95e369c..8a51588c881495f3317e1c3ec79da86db12291da 100644 (file)
@@ -14,6 +14,7 @@ my $pack_errcode;
 my $load_errcode;
 
 my $errcount;
+my $year = (localtime)[5] + 1900;
 
 while (@ARGV) {
        my $arg = $ARGV[0];
@@ -115,7 +116,7 @@ EOF
 }
 
 if($recurse) {
-       @source = (<crypto/*.c>, <crypto/*/*.c>, <ssl/*.c>);
+       @source = ( <crypto/*.c>, <crypto/*/*.c>, <ssl/*.c> )
 } else {
        @source = @ARGV;
 }
@@ -250,7 +251,7 @@ while (($hdr, $lib) = each %libinc)
 
        if ($gotfile) {
          while(<IN>) {
-               if(/^\#define\s+(\S+)\s+(\S+)/) {
+               if(/^\#\s*define\s+(\S+)\s+(\S+)/) {
                        $name = $1;
                        $code = $2;
                        next if $name =~ /^${lib}err/;
@@ -391,7 +392,7 @@ foreach $lib (keys %csrc)
        } else {
            push @out,
 "/* ====================================================================\n",
-" * Copyright (c) 2001-2009 The OpenSSL Project.  All rights reserved.\n",
+" * Copyright (c) 2001-$year The OpenSSL Project.  All rights reserved.\n",
 " *\n",
 " * Redistribution and use in source and binary forms, with or without\n",
 " * modification, are permitted provided that the following conditions\n",
@@ -458,7 +459,8 @@ foreach $lib (keys %csrc)
        print OUT @out;
        undef @out;
        print OUT <<"EOF";
-/* The following lines are auto generated by the script mkerr.pl. Any changes
+/*
+ * The following lines are auto generated by the script mkerr.pl. Any changes
  * made after this point may be overwritten when the script is next run.
  */
 EOF
@@ -472,7 +474,7 @@ EOF
 ${staticloader}void ERR_load_${lib}_strings(void);
 ${staticloader}void ERR_unload_${lib}_strings(void);
 ${staticloader}void ERR_${lib}_error(int function, int reason, char *file, int line);
-#define ${lib}err(f,r) ERR_${lib}_error((f),(r),__FILE__,__LINE__)
+# define ${lib}err(f,r) ERR_${lib}_error((f),(r),__FILE__,__LINE__)
 
 EOF
        }
@@ -483,7 +485,7 @@ EOF
 EOF
 
        foreach $i (@function) {
-               $z=6-int(length($i)/8);
+               $z=48 - length($i);
                if($fcodes{$i} eq "X") {
                        $fassigned{$lib} =~ m/^:([^:]*):/;
                        $findcode = $1;
@@ -497,13 +499,13 @@ EOF
                        $fassigned{$lib} .= "$findcode:";
                        print STDERR "New Function code $i\n" if $debug;
                }
-               printf OUT "#define $i%s $fcodes{$i}\n","\t" x $z;
+               printf OUT "# define $i%s $fcodes{$i}\n"," " x $z;
        }
 
        print OUT "\n/* Reason codes. */\n";
 
        foreach $i (@reasons) {
-               $z=6-int(length($i)/8);
+               $z=48 - length($i);
                if($rcodes{$i} eq "X") {
                        $rassigned{$lib} =~ m/^:([^:]*):/;
                        $findcode = $1;
@@ -517,7 +519,7 @@ EOF
                        $rassigned{$lib} .= "$findcode:";
                        print STDERR "New Reason code   $i\n" if $debug;
                }
-               printf OUT "#define $i%s $rcodes{$i}\n","\t" x $z;
+               printf OUT "# define $i%s $rcodes{$i}\n"," " x $z;
        }
        print OUT <<"EOF";
 
@@ -533,14 +535,21 @@ EOF
        # First, read any existing reason string definitions:
        my %err_reason_strings;
        if (open(IN,"<$cfile")) {
+               my $line = "";
                while (<IN>) {
-                       if (/\b(${lib}_R_\w*)\b.*\"(.*)\"/) {
-                               $err_reason_strings{$1} = $2;
-                       }
-                       if (/\b${lib}_F_(\w*)\b.*\"(.*)\"/) {
-                               if (!exists $ftrans{$1} && ($1 ne $2)) {
-                                       print STDERR "WARNING: Mismatched function string $2\n";
-                                       $ftrans{$1} = $2;
+                       chomp;
+                       $_ = $line . $_;
+                       $line = "";
+                       if (/{ERR_(FUNC|REASON)\(/) {
+                               if (/\b(${lib}_R_\w*)\b.*\"(.*)\"/) {
+                                       $err_reason_strings{$1} = $2;
+                               } elsif (/\b${lib}_F_(\w*)\b.*\"(.*)\"/) {
+                                       if (!exists $ftrans{$1} && ($1 ne $2)) {
+                                               print STDERR "WARNING: Mismatched function string $2\n";
+                                               $ftrans{$1} = $2;
+                                       }
+                               } else {
+                                       $line = $_;
                                }
                        }
                }
@@ -576,14 +585,14 @@ EOF
        print OUT <<"EOF";
 /* $cfile */
 /* ====================================================================
- * Copyright (c) 1999-2009 The OpenSSL Project.  All rights reserved.
+ * Copyright (c) 1999-$year The OpenSSL Project.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer. 
+ *    notice, this list of conditions and the following disclaimer.
  *
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in
@@ -629,7 +638,8 @@ EOF
  *
  */
 
-/* NOTE: this file was auto generated by the mkerr.pl script: any changes
+/*
+ * NOTE: this file was auto generated by the mkerr.pl script: any changes
  * made to it will be overwritten when the script next updates this file,
  * only reason strings will be preserved.
  */
@@ -641,11 +651,10 @@ EOF
 /* BEGIN ERROR CODES */
 #ifndef OPENSSL_NO_ERR
 
-#define ERR_FUNC(func) ERR_PACK($pack_errcode,func,0)
-#define ERR_REASON(reason) ERR_PACK($pack_errcode,0,reason)
+# define ERR_FUNC(func) ERR_PACK($pack_errcode,func,0)
+# define ERR_REASON(reason) ERR_PACK($pack_errcode,0,reason)
 
-static ERR_STRING_DATA ${lib}_str_functs[]=
-       {
+static ERR_STRING_DATA ${lib}_str_functs[] = {
 EOF
        # Add each function code: if a function name is found then use it.
        foreach $i (@function) {
@@ -656,20 +665,22 @@ EOF
                        $fn = $ftrans{$fn};
                }
 #              print OUT "{ERR_PACK($pack_errcode,$i,0),\t\"$fn\"},\n";
-               print OUT "{ERR_FUNC($i),\t\"$fn\"},\n";
+               if(length($i) + length($fn) > 58) {
+                       print OUT "    {ERR_FUNC($i),\n     \"$fn\"},\n";
+               } else {
+                       print OUT "    {ERR_FUNC($i), \"$fn\"},\n";
+               }
        }
        print OUT <<"EOF";
-{0,NULL}
-       };
+    {0, NULL}
+};
 
-static ERR_STRING_DATA ${lib}_str_reasons[]=
-       {
+static ERR_STRING_DATA ${lib}_str_reasons[] = {
 EOF
        # Add each reason code.
        foreach $i (@reasons) {
                my $rn;
                my $rstr = "ERR_REASON($i)";
-               my $nspc = 0;
                if (exists $err_reason_strings{$i}) {
                        $rn = $err_reason_strings{$i};
                } else {
@@ -677,90 +688,87 @@ EOF
                        $rn = $1;
                        $rn =~ tr/_[A-Z]/ [a-z]/;
                }
-               $nspc = 40 - length($rstr) unless length($rstr) > 40;
-               $nspc = " " x $nspc;
-               print OUT "{${rstr}${nspc},\"$rn\"},\n";
+               if(length($i) + length($rn) > 56) {
+                       print OUT "    {${rstr},\n     \"$rn\"},\n";
+               } else {
+                       print OUT "    {${rstr}, \"$rn\"},\n";
+               }
        }
 if($static) {
        print OUT <<"EOF";
-{0,NULL}
-       };
+    {0, NULL}
+};
 
 #endif
 
 ${staticloader}void ERR_load_${lib}_strings(void)
-       {
+{
 #ifndef OPENSSL_NO_ERR
 
-       if (ERR_func_error_string(${lib}_str_functs[0].error) == NULL)
-               {
-               ERR_load_strings($load_errcode,${lib}_str_functs);
-               ERR_load_strings($load_errcode,${lib}_str_reasons);
-               }
+    if (ERR_func_error_string(${lib}_str_functs[0].error) == NULL) {
+        ERR_load_strings($load_errcode, ${lib}_str_functs);
+        ERR_load_strings($load_errcode, ${lib}_str_reasons);
+    }
 #endif
-       }
+}
 EOF
 } else {
        print OUT <<"EOF";
-{0,NULL}
-       };
+    {0, NULL}
+};
 
 #endif
 
 #ifdef ${lib}_LIB_NAME
-static ERR_STRING_DATA ${lib}_lib_name[]=
-        {
-{0     ,${lib}_LIB_NAME},
-{0,NULL}
-       };
+static ERR_STRING_DATA ${lib}_lib_name[] = {
+    {0, ${lib}_LIB_NAME},
+    {0, NULL}
+};
 #endif
 
-
-static int ${lib}_lib_error_code=0;
-static int ${lib}_error_init=1;
+static int ${lib}_lib_error_code = 0;
+static int ${lib}_error_init = 1;
 
 ${staticloader}void ERR_load_${lib}_strings(void)
-       {
-       if (${lib}_lib_error_code == 0)
-               ${lib}_lib_error_code=ERR_get_next_error_library();
+{
+    if (${lib}_lib_error_code == 0)
+        ${lib}_lib_error_code = ERR_get_next_error_library();
 
-       if (${lib}_error_init)
-               {
-               ${lib}_error_init=0;
+    if (${lib}_error_init) {
+        ${lib}_error_init = 0;
 #ifndef OPENSSL_NO_ERR
-               ERR_load_strings(${lib}_lib_error_code,${lib}_str_functs);
-               ERR_load_strings(${lib}_lib_error_code,${lib}_str_reasons);
+        ERR_load_strings(${lib}_lib_error_code, ${lib}_str_functs);
+        ERR_load_strings(${lib}_lib_error_code, ${lib}_str_reasons);
 #endif
 
 #ifdef ${lib}_LIB_NAME
-               ${lib}_lib_name->error = ERR_PACK(${lib}_lib_error_code,0,0);
-               ERR_load_strings(0,${lib}_lib_name);
+        ${lib}_lib_name->error = ERR_PACK(${lib}_lib_error_code, 0, 0);
+        ERR_load_strings(0, ${lib}_lib_name);
 #endif
-               }
-       }
+    }
+}
 
 ${staticloader}void ERR_unload_${lib}_strings(void)
-       {
-       if (${lib}_error_init == 0)
-               {
+{
+    if (${lib}_error_init == 0) {
 #ifndef OPENSSL_NO_ERR
-               ERR_unload_strings(${lib}_lib_error_code,${lib}_str_functs);
-               ERR_unload_strings(${lib}_lib_error_code,${lib}_str_reasons);
+        ERR_unload_strings(${lib}_lib_error_code, ${lib}_str_functs);
+        ERR_unload_strings(${lib}_lib_error_code, ${lib}_str_reasons);
 #endif
 
 #ifdef ${lib}_LIB_NAME
-               ERR_unload_strings(0,${lib}_lib_name);
+        ERR_unload_strings(0, ${lib}_lib_name);
 #endif
-               ${lib}_error_init=1;
-               }
-       }
+        ${lib}_error_init = 1;
+    }
+}
 
 ${staticloader}void ERR_${lib}_error(int function, int reason, char *file, int line)
-       {
-       if (${lib}_lib_error_code == 0)
-               ${lib}_lib_error_code=ERR_get_next_error_library();
-       ERR_PUT_error(${lib}_lib_error_code,function,reason,file,line);
-       }
+{
+    if (${lib}_lib_error_code == 0)
+        ${lib}_lib_error_code = ERR_get_next_error_library();
+    ERR_PUT_error(${lib}_lib_error_code, function, reason, file, line);
+}
 EOF
 
 }
@@ -769,7 +777,7 @@ EOF
        undef %err_reason_strings;
 }
 
-if($debug && defined(%notrans)) {
+if($debug && %notrans) {
        print STDERR "The following function codes were not translated:\n";
        foreach(sort keys %notrans)
        {
@@ -787,7 +795,7 @@ foreach (keys %rcodes) {
        push (@runref, $_) unless exists $urcodes{$_};
 }
 
-if($debug && defined(@funref) ) {
+if($debug && @funref) {
        print STDERR "The following function codes were not referenced:\n";
        foreach(sort @funref)
        {
@@ -795,7 +803,7 @@ if($debug && defined(@funref) ) {
        }
 }
 
-if($debug && defined(@runref) ) {
+if($debug && @runref) {
        print STDERR "The following reason codes were not referenced:\n";
        foreach(sort @runref)
        {