develooper Front page | perl.perl5.changes | Postings from May 2008

Change 33956: Add conditional code to initialise RETVAL, to avoid compiler warnings.

From:
Nicholas Clark
Date:
May 31, 2008 05:00
Subject:
Change 33956: Add conditional code to initialise RETVAL, to avoid compiler warnings.
Change 33956 by nicholas@mouse-mill on 2008/05/31 11:53:20

	Add conditional code to initialise RETVAL, to avoid compiler warnings.
	(There was never an error, as croak() was called before the return).

Affected files ...

... //depot/perl/ext/POSIX/POSIX.xs#155 edit

Differences ...

==== //depot/perl/ext/POSIX/POSIX.xs#155 (text) ====
Index: perl/ext/POSIX/POSIX.xs
--- perl/ext/POSIX/POSIX.xs#154~33936~	2008-05-25 16:47:00.000000000 -0700
+++ perl/ext/POSIX/POSIX.xs	2008-05-31 04:53:20.000000000 -0700
@@ -656,6 +656,11 @@
 	POSIX::WSTOPSIG = 4
 	POSIX::WTERMSIG = 5
     CODE:
+#if !(defined(WEXITSTATUS) || defined(WIFEXITED) || defined(WIFSIGNALED) \
+      || defined(WIFSTOPPED) || defined(WSTOPSIG) || defined (WTERMSIG))
+        RETVAL = 0; /* Silence compilers that notice this, but don't realise
+		       that not_here() can't return.  */
+#endif
 	switch(ix) {
 	case 0:
 #ifdef WEXITSTATUS
End of Patch.



Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About