Re: the key_depends() issue



On Tue, 2011-02-01 at 12:23 +0100, Guillaume Emont wrote:
> Not needed
> ~~~~~~~~~~
> 
> supported_keys() is not strictly needed in a resolution algorithm, and
> could
> only provide a marginal performance improvement if its result is
> cached.
> 


First of all, thanks for the email.


I agree that supported_keys() is not strictly needed to do the job. But
the same can be told about supported_keys() in media sources: they are
not a must to do the job.

Still, I think it is worth to keep supported_keys() to tell applications
which keys are supported, instead of applications needing to call
continuosly may_resolve() to filter which keys are supported. Actually,
I don't see supported_keys() as a complex function.


	J.A.




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]