Re: [PATCH v3 6/6] git-std-lib: add test file to call git-std-lib.a functions
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Sep 15, 2023, 20:22 UTC
- Message-ID
- <xmqqbke3dw4l.fsf@gitster.g>
- In-Reply-To
- <20230915184321.1598611-1-jonathantanmy@google.com>
Jonathan Tan <jonathantanmy@google.com> writes:
Show 19 quoted lines
> Calvin Wan <calvinwan@google.com> writes: >> Add test file that directly or indirectly calls all functions defined in >> git-std-lib.a object files to showcase that they do not reference >> missing objects and that git-std-lib.a can stand on its own. >> >> Certain functions that cause the program to exit or are already called >> by other functions are commented out. >> >> TODO: replace with unit tests >> Signed-off-by: Calvin Wan <calvinwan@google.com> > > I think the TODO should go into the code, so that when we add a unit > test that also deletes stdlib-test.c, we can see what's happening just > from the diff. The TODO should also explain what stdlib-test.c is hoping > to do, and why replacing it is OK. (Also, do we need to invoke all the > functions? I thought that missing functions are checked at link time, or > at the very latest, when the executable is run. No need to change this, > though - invoking all the functions we can is fine.) >
Thanks for excellent reviews (not just against this 6/6 but others, too).