Shourya Shukla <shouryashukla.oo@xxxxxxxxx> writes: >> Following 'enum bisect_error' vocabulary, return variable 'res' is >> always non-positive. >> Let's use '-res' instead of 'abs(res)' to make the code clearer. > >> Mentored-by: Christian Couder <chriscool@xxxxxxxxxxxxx> >> Signed-off-by: Miriam Rubio <mirucam@xxxxxxxxx> >> Helped-by: Junio C Hamano <gitster@xxxxxxxxx> > > Although there is no set rule for this, but I think that your sign-off > goes at the last generally. This is what I have seen on various patches. It is most natural to list these entries in chronological order. Code was written while others helped, and the final seal on what gets sent out is stamped by appending the author's sign-off at the end. Thanks.
