Elijah Newren wrote: > Since both functions are using the same data type, they should either both > refer to it as void *, or both use the real type (struct alloc_state *). > Opt for the latter. > > Reviewed-by: Jonathan Nieder <[email protected]> > Signed-off-by: Elijah Newren <[email protected]>
Not worth rerolling for this, but these usually go in the opposite order to reflect the chronology: Signed-off-by: Elijah Newren <[email protected]> Reviewed-by: Jonathan Nieder <[email protected]> The patch still looks good to me. :)
