Commita76c857

committed

Add comment why seemingly dead code is necessary

1 parentc92be3c commita76c857Copy full SHA for a76c857

File tree

-1

lines changed

-1

lines changed

Lines changed: 9 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -832,7 +832,15 @@ pp_require_safe(pTHX)`
`832`	`832`	`RETPUSHYES;`
`833`	`833`
`834`	`834`	`DIE(aTHX_"Unable to load %s into plperl",name);`
`835`		`-returnNULL;/* keep compiler quiet */`
	`835`	`+/*`
	`836`	`+ * In most Perl versions, DIE() expands to a return statement, so the next`
	`837`	`+ * line is not necessary. But in versions between but not including 5.11.1`
	`838`	`+ * and 5.13.3 it does not, so the next line is necessary to avoid a`
	`839`	`+ * "control reaches end of non-void function" warning from gcc. Other`
	`840`	`+ * compilers such as Solaris Studio will, however, issue a "statement not`
	`841`	`+ * reached" warning instead.`
	`842`	`+ */`
	`843`	`+returnNULL;`
`836`	`844`	`}`
`837`	`845`
`838`	`846`

Comments

(0)