Developers of Sympa

[Matthieu Brunet <address@concealed>]

In &List::new, rather than


if ($list_of_lists{$name}){
        # use the current list in memory and update it
        $liste=$list_of_lists{$name};
}else{
        do_log('debug', 'List object %s created', $name) if $main::opt_d;
##TEMP

        # create a new object list
        bless $liste, $pkg;
}
return undef unless ($liste->load($name));
return $liste;


it seems to be, that one should have 


if ($list_of_lists{$name}){
        # use the current list in memory and update it
        $liste=$list_of_lists{$name};
}else{
        do_log('debug', 'List object %s created', $name) if $main::opt_d;
##TEMP

        # create a new object list
        bless $liste, $pkg;
        return undef unless ($liste->load($name));
}
return $liste;


otherwise, you re-load a list already loaded in the hash array (making
the hash useless). Thus, an unefficiency.

Am I talking nonsense?

Matthieu Brunet