Re: [PATCH] Network::network_equal: fix definition check of array

[Michael Tremer <michael.tremer@ipfire.org>]

[-- Attachment #1: Type: text/plain, Size: 1090 bytes --]

Hello,

this patch shows the following perl errors:

defined(@array) is deprecated at config/cfgroot/network-functions.pl line 114.
	(Maybe you should just omit the defined()?)
defined(@array) is deprecated at config/cfgroot/network-functions.pl line 114.
	(Maybe you should just omit the defined()?)

You can use the testsuite at the bottom of the file to confirm that the
functions are doing what they are supposed to do.

Best,
-Michael

On Mon, 2018-02-12 at 00:19 +0100, Bernhard Held wrote:
> ---
>  config/cfgroot/network-functions.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/config/cfgroot/network-functions.pl b/config/cfgroot/network-functions.pl
> index 2902aabb0..3bf17b092 100644
> --- a/config/cfgroot/network-functions.pl
> +++ b/config/cfgroot/network-functions.pl
> @@ -111,7 +111,7 @@ sub network_equal {
>  	my @bin1 = &network2bin($network1);
>  	my @bin2 = &network2bin($network2);
>  
> -	if (!defined $bin1 || !defined $bin2) {
> +	if (!defined @bin1 || !defined @bin2) {
>  		return undef;
>  	}
>