diff options
author | Jakub Narebski <jnareb@gmail.com> | 2012-02-23 16:54:46 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2012-02-23 12:49:37 -0800 |
commit | 14b289bdf7c31176dc525cbe81173ec636598782 (patch) | |
tree | 8c17e005f4d74f5d630d05afccf338e513b678c1 /bulk-checkin.c | |
parent | Sync with maint (diff) | |
download | tgif-14b289bdf7c31176dc525cbe81173ec636598782.tar.xz |
gitweb: Refactor checking if part of project info need filling
Extract the check if given keys (given parts) of project info needs to
be filled into project_info_needs_filling() subroutine. It is for now
a thin wrapper around "!exists $project_info->{$key}".
Note that !defined was replaced by more correct !exists.
While at it uniquify treating of all project info, adding checks for
'age' field before running git_get_last_activity(), and also checking
for all keys filled in code protected by conditional, and not only
one.
The code now looks like this
foreach my $project (@$project_list) {
if (given keys need to be filled) {
fill given keys
}
...
}
Signed-off-by: Jakub Narebski <jnareb@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'bulk-checkin.c')
0 files changed, 0 insertions, 0 deletions