initial gencache implementation
abartlet at dp.samba.org
abartlet at dp.samba.org
Fri Sep 6 02:11:00 GMT 2002
On Fri, Sep 06, 2002 at 10:17:19AM +1000, Tim Potter wrote:
> On Thu, Sep 05, 2002 at 12:15:36PM +0200, Rafal Szczesniak wrote:
>
> > This is first implementation of caching mechanism. It includes
> > both lib/gencache.c code and utils/net_cache.c as command-line
> > control/testing tool.
> >
> > comments are welcome
>
> Rafal, that looks pretty good. Since you ask, I do have a few comments.
> (-:
>
> You assume that any cached data will be in null terminated string format
> which is not always the case.
I understand this is a design property - it's up to the caller to mess with
structs etc. This keeps the cache simple, and allows for human inspection
with 'net cache' etc.
> This is just my personal opinion but I would prefer for gencache_set to
> crash if you pass it a NULL pointer for the key or value parameter.
> Returning false in this case only hides the error until further along
> in the execution path by which time it will be more difficult to track
> down the original error.
Agreed.
Andrew Bartlett
More information about the samba-technical
mailing list