Re: [PATCH] revision.c: Correctly dereference interesting_cache

2015-06-19 Thread Jonathan Nieder
Hi, Stefan Beller wrote: This was introduced at b6e8a3b5 (2015-04-17, limit_list: avoid quadratic behavior from still_interesting), which also introduced the check a few lines before, which already dereferences `interesting_cache`. So at this point `interesting_cache` is guaranteed to be

Re: [PATCH] revision.c: Correctly dereference interesting_cache

2015-06-19 Thread Jeff King
On Fri, Jun 19, 2015 at 12:01:23PM -0700, Stefan Beller wrote: This was introduced at b6e8a3b5 (2015-04-17, limit_list: avoid quadratic behavior from still_interesting), which also introduced the check a few lines before, which already dereferences `interesting_cache`. So at this point

[PATCH] revision.c: Correctly dereference interesting_cache

2015-06-19 Thread Stefan Beller
This was introduced at b6e8a3b5 (2015-04-17, limit_list: avoid quadratic behavior from still_interesting), which also introduced the check a few lines before, which already dereferences `interesting_cache`. So at this point `interesting_cache` is guaranteed to be not NULL. The code is called