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.
>
> While making this change, simplify the error handling in cache_get
> with a new "goto FAIL" block so the cleanup code exists in a single
> place, rather than being spread throughout the function over and over.

Is there some reason we should do this as a separate patch instead of
just squashing it?
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to