Re: [PATCH 2/3] Move copy_note_for_rewrite + friends from builtin/notes.c to notes-utils.c
- From
Felipe Contreras <felipe.contreras@gmail.com>
- Date
- Jun 12, 2013, 18:28 UTC
- Message-ID
- <CAMP44s3KAeDPo1Cw8eFsU=A6H7oUGmf+eLAMvGV+R2_hPXHLbw@mail.gmail.com>
- In-Reply-To
- <CALKQrgfxrKz5bB=AAmL1ZtBFRK2Bx6TrRd1AsMEVv8bTAH0KCg@mail.gmail.com>
On Wed, Jun 12, 2013 at 2:10 AM, Johan Herland <johan@herland.net> wrote:
Show 23 quoted lines
> On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras > <felipe.contreras@gmail.com> wrote: >> On Tue, Jun 11, 2013 at 7:13 PM, Johan Herland <johan@herland.net> wrote: >>> This is a pure code movement of the machinery for copying notes to >>> rewritten objects. This code was located in builtin/notes.c for >>> historical reasons. In order to make it available to builtin/commit.c >>> it was declared in builtin.h. This was more of an accident of history >>> than a concious design, and we now want to make this machinery more >>> widely available. >>> >>> Hence, this patch moves the code into the new notes-utils.[hc] files >>> which are included into libgit.a. Except for adjusting #includes >>> accordingly, this patch merely moves the relevant functions verbatim >>> into the new files. >>> >>> Cc: Thomas Rast <trast@inf.ethz.ch> >>> Signed-off-by: Johan Herland <johan@herland.net> >> >> I wonder where you got that idea from. Did you come up with that out thin air? > > Obviously not. I should add > > Suggested-by: Junio C Hamano <gitster@pobox.com>
You are still not explaining where the idea came from. And you are doing that with the express purpose of annoying.
Where did the idea come from?
-- Felipe Contreras