[prev in list] [next in list] [prev in thread] [next in thread]
List: git
Subject: Re: [PATCH v13 03/14] Move libified code from builtin/apply.c to apply.{c,h}
From: Stefan Beller <sbeller () google ! com>
Date: 2016-08-31 21:57:53
Message-ID: CAGZ79kb0gao2KJFi9X3V2ME+1-xnbuSNhjC14Vnn3oP-1ac35A () mail ! gmail ! com
[Download RAW message or body]
On Sat, Aug 27, 2016 at 11:45 AM, Christian Couder
<christian.couder@gmail.com> wrote:
> As most of the apply code in builtin/apply.c has been libified by a numbe=
r of
> previous commits, it can now be moved to apply.{c,h}, so that more code c=
an
> use it.
>
> Helped-by: Nguy=E1=BB=85n Th=C3=A1i Ng=E1=BB=8Dc Duy <pclouds@gmail.com>
> Helped-by: Ramsay Jones <ramsay@ramsayjones.plus.com>
> Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
> ---
> apply.c | 4731 +++++++++++++++++++++++++++++++++++++++++++++++++=
+++++
> apply.h | 19 +
> builtin/apply.c | 4733 +------------------------------------------------=
------
I deduce by roughly the same line count in the .c files, it is just
moving files over.
(On my todo list I have an idea how to make reviewing patches like this eas=
ier.)
> 3 files changed, 4751 insertions(+), 4732 deletions(-)
>
> diff --git a/apply.c b/apply.c
> index 2eac3e3..7b96130 100644
> --- a/apply.c
> +++ b/apply.c
> @@ -1,5 +1,23 @@
> +/*
> + * apply.c
> + *
> + * Copyright (C) Linus Torvalds, 2005
We're very inconsistent with the intellectual property log.
Sometimes we have that at the top of a file, sometimes we don't
and rather point at the git history to find out who touched the code.
I'd rather use the history instead of having a bunch of copyright lines.
So maybe consider to drop this introductory comment as a
{preparatory, follow up} cleanup?
(This is also a nit that doesn't require a reroll on its own)
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic