On Thu, Aug 18, 2016 at 10:56 AM, Stefan Beller <sbeller@xxxxxxxxxx> wrote:
> On Wed, Aug 17, 2016 at 5:51 PM, Jacob Keller <jacob.e.keller@xxxxxxxxx> wrote:
>> - if (!graph)
>> + if (!graph) {
>> + graph_show_line_prefix(default_diffopt);
>> return;
>> + }
>
>> - if (graph_is_commit_finished(graph))
>> + if (graph_is_commit_finished(graph)) {
>> + graph_show_line_prefix(&graph->revs->diffopt);
>> return 0;
>> + }
>
> This seems to be a reoccurring pattern, i.e. we need to add a lot of
> one off instructions before a return. Is it possible to make the call
> earlier instead
> of just before the returns? (or later for that matter) ?
>
Not sure what you mean by this...? Just move the call up above this? I
think that will work, let me give it a try.
Thanks,
Jake
>
> Thanks,
> Stefan
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html