On Wed, Feb 04, 2009 at 02:20:46PM -0800, [email protected] wrote:

> > What about
> > 
> >     if refresh_catalog or purge_catalog:
> >         destroy
> >         destroy
> > 
> >         if refresh_catalog and refresh_allowed:
> >             retrieve
> > 
> >         cache
> 
> Okay, one further question about tidiness, since Shawn brought up the
> case where an authority has its origin changed and is simultaneously
> disabled.
> 
> Do you prefer:
> 
>               if disabled:
>                       purge_catalog = True
>                       refresh_catalog = False
>               else:
>                       refresh_catalog = True
> 
>       if refresh_catalog or purge_catalog:
>               destroy
>               destroy
> 
>               if refresh_catalog and refresh_allowed:
>                       retrieve
>               else:
>                       cache
> 
> or:
> 
>               if disabled:
>                       purge_catalog = True
>               else:
>                       refresh_catalog = True
> 
>       if refresh_catalog or purge_catalog:
>               destroy
>               destroy
> 
>               if purge_catalog:
>                       cache
>               elif refresh_allowed:
>                       retrieve

The latter, I guess, though I don't understand how the if clause differs
between the two examples, assuming that both purge_catalog and
refresh_catalog start out False.

Danek
_______________________________________________
pkg-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/pkg-discuss

Reply via email to