Re: [PATCH 4/6] introduce a commit metapack
- From
Jeff King <peff@peff.net>
- Date
- Jan 30, 2013, 07:12 UTC
- Message-ID
- <20130130071248.GE11147@sigill.intra.peff.net>
- In-Reply-To
- <CACsJy8COCpr_CvQPhBFtVnsKztA3xgo01=zKG4WPcFfCbNTmgw@mail.gmail.com>
On Wed, Jan 30, 2013 at 10:36:10AM +0700, Nguyen Thai Ngoc Duy wrote:
Show 9 quoted lines
> On Tue, Jan 29, 2013 at 4:16 PM, Jeff King <peff@peff.net> wrote:
> > +int commit_metapack(unsigned char *sha1,
> > + uint32_t *timestamp,
> > + unsigned char **tree,
> > + unsigned char **parent1,
> > + unsigned char **parent2)
> > +{
>
> Nit picking. tree, parent1 and parent2 can/should be "const unsigned char **".Thanks, I'll fix it in the next version.