Since v3, I have taken Junio's suggestion to replace '@' with 'HEAD' at the beginning of run_add_p(). Due to this, I no longer think we need to have/export the_user_meant_head() function just for the single instance in builtin/checkout.c. Therefore, I have hardcoded the '@' condition in builtin/checkout.c. Also, Phillip mentioned that the PERL prerequisites in the test files that are touched by this series are unnecessary and therefore I have attached a patch to remove them. Ghanshyam Thakkar (3): add-patch: remove unnecessary NEEDSWORK comment add-patch: classify '@' as a synonym for 'HEAD' add -p tests: remove Perl prerequisite add-patch.c | 12 ++++------ builtin/checkout.c | 11 +++++----- t/t2016-checkout-patch.sh | 46 ++++++++++++++++++++++----------------- t/t2071-restore-patch.sh | 46 +++++++++++++++++++++------------------ t/t7105-reset-patch.sh | 32 +++++++++++++++++---------- 5 files changed, 82 insertions(+), 65 deletions(-) -- 2.43.0
