From: Robin Barker <RMBarker@cpan.org>
Date: Fri, 7 Sep 2001 18:28:23 +0000 (+0100)
Subject: Silence the remaing format warning
X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=b3fe48277ce10a58a036b1303b4d9ed1e50428b1;p=p5sagit%2Fp5-mst-13.2.git

Silence the remaing format warning
Message-Id: <200109071728.SAA13569@tempest.npl.co.uk>

p4raw-id: //depot/perl@11941
---

diff --git a/Porting/pumpkin.pod b/Porting/pumpkin.pod
index 0ef144a..a816c48 100644
--- a/Porting/pumpkin.pod
+++ b/Porting/pumpkin.pod
@@ -729,15 +729,33 @@ produced by this process.
 
 A more accurate approach is the following commands:
 
-    sh Configure -des -Dccflags=-Wformat ...
-    make miniperl		# without -DCHECK_FORMAT
-    perl -i.orig -pwe 's/-Wformat/-DCHECK_FORMAT $&/' config.sh
-    sh Configure -S
-    make >& make.log		# build from correct miniperl
+=over 4
+
+=item *
+
+build miniperl with -DCHECK_FORMAT 
+
+    make clean
+    make miniperl OPTIMIZE=-DCHECK_FORMAT >& mini.log	
+
+=item *
+
+build a clean miniperl,
+and build everything else from that with -DCHECK_FORMAT
+
     make clean
-    make miniperl >& mini.log	# build miniperl with -DCHECK_FORMAT 
-    perl -nwe 'print if /^\S+:/ and not /^make\b/' mini.log make.log
+    make miniperl	
+    make all OPTIMIZE=-DCHECK_FORMAT >& make.log  
+		
+=item *
+
+clean up, and print warnings from the log files
+
     make clean
+    perl -nwe 'print if /^\S+:/ and not /^make\b/' \
+	mini.log make.log
+
+=back
 
 (-Wformat support by Robin Barker.)
 
diff --git a/mg.c b/mg.c
index ef6963d..9def37f 100644
--- a/mg.c
+++ b/mg.c
@@ -2317,7 +2317,7 @@ Perl_sighandler(int sig)
 	(void)rsignal(sig, &Perl_csighandler);
 #endif
 #endif /* !PERL_MICRO */
-	Perl_die(aTHX_ Nullch);
+	Perl_die(aTHX_ Nullformat);
     }
 cleanup:
     if (flags & 1)
diff --git a/perl.h b/perl.h
index 8655111..0610ae7 100644
--- a/perl.h
+++ b/perl.h
@@ -2020,6 +2020,14 @@ typedef pthread_key_t	perl_key;
 #  endif
 #endif
 
+#ifndef Nullformat
+#  ifdef CHECK_FORMAT
+#    define Nullformat "%s",""
+#  else
+#    define Nullformat Nullch
+#  endif
+#endif
+
 /* Some unistd.h's give a prototype for pause() even though
    HAS_PAUSE ends up undefined.  This causes the #define
    below to be rejected by the compiler.  Sigh.
diff --git a/pp_sys.c b/pp_sys.c
index bf32b3c..70b1660 100644
--- a/pp_sys.c
+++ b/pp_sys.c
@@ -468,7 +468,7 @@ PP(pp_die)
 		    sv_setsv(error,*PL_stack_sp--);
 		}
 	    }
-	    DIE(aTHX_ Nullch);
+	    DIE(aTHX_ Nullformat);
 	}
 	else {
 	    if (SvPOK(error) && SvCUR(error))