Am 20.11.2017 um 21:39 schrieb Stefan Beller: > On Sat, Nov 18, 2017 at 10:08 AM, René Scharfe <l.s.r@xxxxxx> wrote: >> Non-empty lines before a function definition are most likely comments >> for that function and thus relevant. Include them in function context. >> >> Such a non-empty line might also belong to the preceding function if >> there is no separating blank line. Stop extending the context upwards >> also at the next function line to make sure only one extra function body >> is shown at most. > > Can we add another heuristic, that checks for common function body ends, e.g. > if the preceding line contains '}' but is not commented (the line doesn't > contain '*/' '//', '#'), we have a strong hint that it is a function, not an > additional comment. C comments containing "}" as part of the text would only be shown partially, e.g: /* * Not shown because of the curly closing brace in ${PATH}. * Shown. */ Two examples in git's repo are in refs.h and sha1-lookup.c. Before diving deeper: Is it worth it? Does the heuristic in this series produce excessive context often? Enough to be annoying? Thanks, René