On 07/14/2016 10:08 PM, Eric Blake wrote:
On 07/14/2016 07:10 AM, Cao jin wrote:
replace tab with spaces
Signed-off-by: Cao jin <caoj.f...@cn.fujitsu.com>
---
async.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Whitespace-only changes are best done as part of a series that is
already touching nearby code for other reasons (depending on the size of
the whitespace changes and on the rest of your patch, it may be okay to
squash the whitespace change in place, or better to split into separate
patches to make review of both patches easier). Otherwise, it just
makes 'git blame' output dirtier.
I see.
Since async.c & aio-posix.c are belong to the same maintaiers, so, Fam &
Stefan, is it ok to squash this into that "remove useless parameter"
patch? If not, we can just forget this one.
--
Yours Sincerely,
Cao jin
