Re: [PATCH v2 5/8] status: print per-file porcelain v2 status data

2016-07-26 Thread Jeff Hostetler
On 07/25/2016 04:23 PM, Junio C Hamano wrote: Jeff Hostetler writes: +static void wt_porcelain_v2_print(struct wt_status *s); + There is no point in this forward declaration, if you just place the implementation of these functions here, no? Right. I just did it that way to make the diffs

Re: [PATCH v2 5/8] status: print per-file porcelain v2 status data

2016-07-25 Thread Junio C Hamano
Jeff Hostetler writes: > +static void wt_porcelain_v2_print(struct wt_status *s); > + There is no point in this forward declaration, if you just place the implementation of these functions here, no? > +/* > + * Print porcelain v2 info for tracked entries with changes. > + */ > +static void wt_p