Re: [PATCH 2/3] Move copy_note_for_rewrite + friends from builtin/notes.c to notes-utils.c
- From
Johan Herland <johan@herland.net>
- Date
- Jun 12, 2013, 07:10 UTC
- Message-ID
- <CALKQrgfxrKz5bB=AAmL1ZtBFRK2Bx6TrRd1AsMEVv8bTAH0KCg@mail.gmail.com>
- In-Reply-To
- <CAMP44s2pUW_+w6B_R-A=vxOg1Ay6iLmc4MQsA_sfDF+GP-XsWw@mail.gmail.com>
On Wed, Jun 12, 2013 at 2:32 AM, Felipe Contreras <felipe.contreras@gmail.com> wrote:
Show 17 quoted lines
> 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>
...Johan
-- Johan Herland, <johan@herland.net> www.herland.net