Confirmed, although && binds more tightly than list operators rightward, so strictly speaking that should be either:
if (defined($ENV{'DEBEMAIL'}) && $ENV{'DEBEMAIL'} =~ /ubuntu/) {
... or:
if (defined $ENV{'DEBEMAIL'} and $ENV{'DEBEMAIL'} =~ /ubuntu/) {
The prevailing style in this file leans towards the former.
(In practice it happens to make no difference in this case since undef is also false, but better safe than sorry.)
Confirmed, although && binds more tightly than list operators rightward, so strictly speaking that should be either:
if (defined( $ENV{'DEBEMAIL' }) && $ENV{'DEBEMAIL'} =~ /ubuntu/) {
... or:
if (defined $ENV{'DEBEMAIL'} and $ENV{'DEBEMAIL'} =~ /ubuntu/) {
The prevailing style in this file leans towards the former.
(In practice it happens to make no difference in this case since undef is also false, but better safe than sorry.)