Add comment why seemingly dead code is necessary

This commit is contained in:
Peter Eisentraut 2012-07-16 22:08:04 +03:00
parent c92be3c059
commit a76c857eba

View File

@ -832,7 +832,15 @@ pp_require_safe(pTHX)
RETPUSHYES;
DIE(aTHX_ "Unable to load %s into plperl", name);
return NULL; /* keep compiler quiet */
/*
* In most Perl versions, DIE() expands to a return statement, so the next
* line is not necessary. But in versions between but not including 5.11.1
* and 5.13.3 it does not, so the next line is necessary to avoid a
* "control reaches end of non-void function" warning from gcc. Other
* compilers such as Solaris Studio will, however, issue a "statement not
* reached" warning instead.
*/
return NULL;
}