On Wed, Jan 6, 2016 at 2:42 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Karthik Nayak <karthik.188@xxxxxxxxx> writes: > >> In align_atom_parser() the 'width' variable is an int, which requires >> an explicit type conversion when used in strtoul_ui(). Hence make it >> an unsigned int. >> >> Helped-by: Eric Sunshine <sunshine@xxxxxxxxxxxxxx> >> Signed-off-by: Karthik Nayak <Karthik.188@xxxxxxxxx> >> --- > > Shouldn't this be done in [09/15] from the beginning? As it's not > like [09/15] was a pure code movement, this step looks like "Oops, > I screwed up earlier, and here is a fixup". > > If we want to do this as a separate step, it would be easier to > follow if it is done _before_ 09/15 to the original of the code > movement, I think. > Haha, does seem that way. Will merge it in. -- Regards, Karthik Nayak -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html