Developers of Sympa

[Olivier Salaun - CRU <address@concealed>]

Matthieu Brunet wrote :
> In &List::new, rather than
> [...]
> }
> return undef unless ($liste->load($name));
> [...]
> it seems to be, that one should have 
> [...]
>       return undef unless ($liste->load($name));
> }
> [...]
> otherwise, you re-load a list already loaded in the hash array (making
> the hash useless). Thus, an unefficiency.

List::load() loads the config file only if it hasn't been done or 
if it has changed on disk. 
It also updates subscribers if "user_data_source" is  "file" or "include".
That's why List::new() calls it anyhow.

It's quite efficient this way.

--
Olivier Salaun
Comite Reseaux des Universites