Re: [PATCH v2 1/2] commit: fix --short and --porcelain options

2018-05-02 Thread Samuel Lijin
On Tue, May 1, 2018 at 10:50 PM Junio C Hamano wrote: > Samuel Lijin writes: > > Mark the commitable flag in the wt_status object in the call to > > `wt_status_collect()`, instead of in `wt_longstatus_print_updated()`, > > and simplify the logic in the latter function to take advantage of the >

Re: [PATCH v2 1/2] commit: fix --short and --porcelain options

2018-05-01 Thread Junio C Hamano
Samuel Lijin writes: > Mark the commitable flag in the wt_status object in the call to > `wt_status_collect()`, instead of in `wt_longstatus_print_updated()`, > and simplify the logic in the latter function to take advantage of the > logic shifted to the former. This means that callers do not nee

[PATCH v2 1/2] commit: fix --short and --porcelain options

2018-04-30 Thread Samuel Lijin
Mark the commitable flag in the wt_status object in the call to `wt_status_collect()`, instead of in `wt_longstatus_print_updated()`, and simplify the logic in the latter function to take advantage of the logic shifted to the former. This means that callers do not need to use `wt_longstatus_print_u