Re: [RFC PATCH 1/1] completion: load completion file for external subcommand
- Date: Tue, 10 Apr 2018 11:37:49 +0200
- From: Florian Gamböck <mail@xxxxxxxx>
- Subject: Re: [RFC PATCH 1/1] completion: load completion file for external subcommand
On 2018-04-10 07:06, Junio C Hamano wrote:
Florian Gamböck <mail@xxxxxxxx> writes:
Does my reasoning make sense?
Not at all.
:-( That actually hurt a bit.
I mean, the result will be exactly the same, we are clearly only
talking about readability here.
... I agree, and I also think the "readability" is not absolute
FWIW, personally I'd find "if $completion_func does not yet exist and
a way to dynload stuff exists, then use that way to load it from an
external file" the most natural way to express what you are doing.
Then let's go with that. Maybe I am overthinking things here and
re-wording doesn't hurt.
I'll send a new patch version later this day, including the commit
message re-wording suggestion from Stefan.
Thank you for your feedback!