On Mon, Sep 25, 2017 at 1:20 AM, Kaartic Sivaraam <kaartic.sivaraam@xxxxxxxxx> wrote: > builtin/branch.c | 41 +++++++++++++++++++++++++++++++++++------ > 1 file changed, 35 insertions(+), 6 deletions(-) The code of 4 and 5 looks good to me, though a small nit below. Thanks, Stefan > +static void get_error_msg(struct strbuf* error_msg, const char* oldname, unsigned old_branch_exists, > + const char* newname, int new_branch_validation_result) nit here and in the return of validate_branch_creation: It would be clearer if this is not just 'int', but actually spelling out that it is the enum.
