Re: [PATCH 1/5] gitweb: Option for filling only specified info in fill_project_list_info
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Feb 10, 2012, 17:43 UTC
- Message-ID
- <7v1uq2shwg.fsf@alter.siamese.dyndns.org>
- In-Reply-To
- <201202101456.45944.jnareb@gmail.com>
Jakub Narebski <jnareb@gmail.com> writes:
Show 6 quoted lines
> (with either "..., 'owner', ..." or "..., [ 'owner' ], ..." for single-key
> filling), or
>
> if (project_info_needs_filled($pr, ['age', 'age_string'], \@fill_only) {
>
> Is it?Whatever. I am not sure what @fill_only is needed for, if the name stands for "only fill these fields, if this argument is empty". After all, doesn't the above example callsite, without ", \@fill_only" at the end, say "I am going to use age and age_string, so these two need to be filled" already?