From: Stefan Beller <sbeller@google.com>
Date: Mon, 05 Feb 2018 18:57:39 GMT
Subject: Re: [PATCH v7 12/31] merge-recursive: move the get_renames() function
Message-ID: <CAGZ79kbhieX+ZkG-Tv_NNJMUw0=syDthnW1Q=o4jhotaTiC0Ow@mail.gmail.com>
In-Reply-To: <CABPp-BFDgDDa_fPSFJQUSzR1k5-ix0SWrviUPFu+SCoyWfG5cQ@mail.gmail.com>

> Move this function so it can re-use some others (without either
> moving all of them or adding an annoying split between function
> declarations and definitions).  Cheat slightly by adding a blank line
> for readability, and in order to silence checkpatch.pl.
>
> Does that sound better?

Yeah, that sounds perfect (no red flags on reading)

Thanks!