[Mesa-dev] [PATCH 3/7] glsl: Add initial functions to implement an on-disk cache
cworth at cworth.org
Wed Feb 4 16:14:22 PST 2015
On Wed, Feb 04 2015, Kenneth Graunke wrote:
> The cache will need to be much larger than 1024 entries - perhaps by an
> order of magnitude.
Thanks for the feedback. I had meant to add a comment next to that 1024
in the code along the lines of "This value was chosen arbitrarily. An
appropriate value will need to be found with testing."
> We should respect $XDG_CACHE_HOME from the XDG base directory spec:
> Also, the snprintf makes me a bit nervous
See patch 5/7. It fixes both of these problems.
(My apologies for setting you up to review code that was replaced later
in the series---but I'm always a little hesitant to squash things to
much when only some of the code is actually mine).
> Typo: unnecessary compile (missing 'n'). Also, */ goes on its own
Thanks. I've fixed both of these now.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 818 bytes
Desc: not available
More information about the mesa-dev