Ghanshyam Thakkar <shyamthakkar001@xxxxxxxxx> writes:
> helper/test-oidmap.c along with t0016-oidmap.sh test the oidmap.h
> library which is built on top of hashmap.h.
>
> Migrate them to the unit testing framework for better performance,
> concise code and better debugging. Along with the migration also plug
> memory leaks and make the test logic independent for all the tests.
> The migration removes 'put' tests from t0016, because it is used as
> setup to all the other tests, so testing it separately does not yield
> any benefit.
>
> Helped-by: Junio C Hamano <gitster@xxxxxxxxx>
> Helped-by: Phillip Wood <phillip.wood123@xxxxxxxxx>
> Reviewed-by: Josh Steadmon <steadmon@xxxxxxxxxx>
> Mentored-by: Christian Couder <chriscool@xxxxxxxxxxxxx>
> Mentored-by: Kaartic Sivaraam <kaartic.sivaraam@xxxxxxxxx>
> Signed-off-by: Ghanshyam Thakkar <shyamthakkar001@xxxxxxxxx>
The trailer lines should come more-or-less in chronological order.
I do not know exact sequence of events, but mentoring would have
happened first before the initial iteration was posted, then Phillip
helped to improve it during iterations, the latest was reviewed by
Josh and then you folded the little test improvement from me and
Phillip? And to seal the whole thing off, you add your sign-off at
the end.
Technically speaking, any change after a review invalidates an
earlier "Reviewed-by", but the updates we see here, relative to the
iteration that received the "Reviewed-by", are not significant or
large enough to warrant that, so let's pretend that Josh would be
happy with the end result, even with our little additions.
Which leads us to the trailer lines ordered like so:
Mentored-by: Christian Couder <chriscool@xxxxxxxxxxxxx>
Mentored-by: Kaartic Sivaraam <kaartic.sivaraam@xxxxxxxxx>
Helped-by: Phillip Wood <phillip.wood123@xxxxxxxxx>
Helped-by: Junio C Hamano <gitster@xxxxxxxxx>
Reviewed-by: Josh Steadmon <steadmon@xxxxxxxxxx>
Signed-off-by: Ghanshyam Thakkar <shyamthakkar001@xxxxxxxxx>
The changes since the previous round look exactly as expected.
Looking very good.
Will queue. Let's mark it for 'next'.
Thanks.
