On Tue, 2012-01-10 at 14:33 -0500, Simo Sorce wrote:
On Tue, 2012-01-10 at 10:15 -0500, Simo Sorce wrote:
> > Sure, we can talk about it. I'm looking at it from the users'
> > perspectives, who I think would generally expect (and be alright
> with)
> > the fast cache being emptied on service restart. Since we still have
> the
> > not-quite-as-fast persistent LDB cache, I think the gain isn't worth
> the
> > user confusion.
>
> Ok, I think I can understand this, it will also simplify the code I
> guess so I'll change the patch.
Attached new set of patches.
I changed the init code to always create a new cache file on restart,
incidentally this also got rid of the stat() call, so that problem has
been put to rest permanently :)
I also changed all functions to use errno_t as the return type in their
declaration when they return a errno code.
I think all the suggestions seen in the thread so far have been
implemented as requested at this point, with the exception of the part
where I should implement and use new options as that required additional
code instead of mere refactoring. I will add those features in a later
rebase.
Oops had a little erro converting to use errno, new patch attached to
fix just that, all other patches should be fine.
Simo.
--
Simo Sorce * Red Hat, Inc * New York