Rafael Garcia-Suarez wrote on 2009-11-06:
> 2009/11/2 Gerard Goossen <gerard@ggoossen.net>:
>> DIE does exit, it just does do that using the normal control flow.
>> Just reading the patch again, I think I understand your confusion:
The
>> "return NORMAL" added to some of the pp_* functions are useless, they
>> are never actually executed, they are only there to suppress some
>> compiler warnings about functions which don't have a return
statement.
>>
>> Does this resolve your concerns?
>
> Yes.
> Thanks for the patch and explanations, applied as
> bb4c52e023e0fcade469e46e9a1d5245ff44f3f2
Is it this that has just caused all these warnings?
cl -c -nologo -GF -W3 -I..\lib\CORE -I.\include -I. -I.. -DWIN32
-D_CONSOLE -DNO_STRICT -DHAVE_DES_FCRYPT -D_CRT_SECURE_NO_DEPRECATE
-D_CRT_NONSTDC_NO_DEPRECATE -DPERLDLL -DPERL_CORE -MD -Zi -DNDEBUG -O1
-DPERL_EXTERNAL_GLOB -DPERL_IS_MINIPERL -Fo.\mini\pp_sys.obj ..\pp_sys.c
pp_sys.c
c:\gitclients\perl\pp_sys.c(3561) : warning C4716: 'Perl_pp_chroot' :
must return a value
c:\gitclients\perl\pp_sys.c(4062) : warning C4716: 'Perl_pp_fork' : must
returna value
c:\gitclients\perl\pp_sys.c(4325) : warning C4716: 'Perl_pp_getppid' :
must return a value
c:\gitclients\perl\pp_sys.c(4346) : warning C4716: 'Perl_pp_getpgrp' :
must return a value
c:\gitclients\perl\pp_sys.c(4379) : warning C4716: 'Perl_pp_setpgrp' :
must return a value
c:\gitclients\perl\pp_sys.c(4392) : warning C4716: 'Perl_pp_getpriority'
: must return a value
c:\gitclients\perl\pp_sys.c(4407) : warning C4716: 'Perl_pp_setpriority'
: must return a value
c:\gitclients\perl\pp_sys.c(4622) : warning C4716: 'Perl_pp_semget' :
must return a value
c:\gitclients\perl\pp_sys.c(4830) : warning C4716: 'Perl_pp_gnetent' :
must return a value
c:\gitclients\perl\pp_sys.c(4976) : warning C4716: 'Perl_pp_shostent' :
must return a value
c:\gitclients\perl\pp_sys.c(4987) : warning C4716: 'Perl_pp_snetent' :
must return a value
c:\gitclients\perl\pp_sys.c(4998) : warning C4716: 'Perl_pp_sprotoent' :
must return a value
c:\gitclients\perl\pp_sys.c(5009) : warning C4716: 'Perl_pp_sservent' :
must return a value
c:\gitclients\perl\pp_sys.c(5021) : warning C4716: 'Perl_pp_ehostent' :
must return a value
c:\gitclients\perl\pp_sys.c(5033) : warning C4716: 'Perl_pp_enetent' :
must return a value
c:\gitclients\perl\pp_sys.c(5045) : warning C4716: 'Perl_pp_eprotoent' :
must return a value
c:\gitclients\perl\pp_sys.c(5057) : warning C4716: 'Perl_pp_eservent' :
must return a value
c:\gitclients\perl\pp_sys.c(5290) : warning C4716: 'Perl_pp_gpwent' :
must return a value
c:\gitclients\perl\pp_sys.c(5301) : warning C4716: 'Perl_pp_spwent' :
must return a value
c:\gitclients\perl\pp_sys.c(5312) : warning C4716: 'Perl_pp_epwent' :
must return a value
c:\gitclients\perl\pp_sys.c(5386) : warning C4716: 'Perl_pp_ggrent' :
must return a value
c:\gitclients\perl\pp_sys.c(5397) : warning C4716: 'Perl_pp_sgrent' :
must return a value
c:\gitclients\perl\pp_sys.c(5408) : warning C4716: 'Perl_pp_egrent' :
must return a value
c:\gitclients\perl\pp_sys.c(5521) : warning C4716: 'Perl_pp_syscall' :
must return a value
Thread Previous
|
Thread Next