The user can put the cache directory anywhere, so it's not safe to use
fixed-size arrays to store filenames. Instead, allocate the cache
pointer itself as a ralloc context and use that to dynamically
allocate all filenames.
While making this change, simplify the error handling in cache_get
with a
On Wed, Feb 4, 2015 at 1:53 PM, Carl Worth cwo...@cworth.org wrote:
The user can put the cache directory anywhere, so it's not safe to use
fixed-size arrays to store filenames. Instead, allocate the cache
pointer itself as a ralloc context and use that to dynamically
allocate all filenames.
On Wed, Feb 04 2015, Matt Turner wrote:
Is there some reason we should do this as a separate patch instead of
just squashing it?
Not really. I've just squashed it, (while also fixing the space before
the parenthesis---I should probably add a pre-commit check for those to
help me break that