diff options
| author | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2017-04-29 17:00:09 +0200 |
|---|---|---|
| committer | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2017-04-29 19:18:50 +0200 |
| commit | 146258f82fc15d0a5f8779447a4919429496e9e3 (patch) | |
| tree | 62a263c4bd6f89e12c33c1aa152bed46e2cb874b /bin/lo-commit-stat | |
| parent | ad3e1afe5969d6c53b1ed285b7c36685bd457b1c (diff) | |
lo-commit-stat: drop unnecessary check whether hash is defined
besides using deprecated syntax – "if (defined %hash)" (or "if (defined
@array)" for that matter) – just using "if (%hash)" (or "if (@array)")
would do, it is also unnecessary to create the hash if it doesn't exist
(autovivification will take care of that)
Change-Id: Ib5704b2d609ae3658f3c166f9e26a051957149f8
Diffstat (limited to 'bin/lo-commit-stat')
| -rwxr-xr-x | bin/lo-commit-stat | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/bin/lo-commit-stat b/bin/lo-commit-stat index e33ca798f41e..a4e3f7d1ae32 100755 --- a/bin/lo-commit-stat +++ b/bin/lo-commit-stat @@ -68,7 +68,6 @@ sub search_bugs($$$$) # someone mistyped fdo as fd0 $bug =~ s/^fd0\#/fdo#/; # save the bug number - %{$pdata->{$module}{$commit_id}{'bugs'}} = () if (! defined %{$pdata->{$module}{$commit_id}{'bugs'}}); $pdata->{$module}{$commit_id}{'bugs'}{$bug} = 1; } @@ -262,7 +261,6 @@ sub get_bug_list($$$) foreach my $module ( keys %{$pdata}) { foreach my $id ( keys %{$pdata->{$module}}) { foreach my $bug (keys %{$pdata->{$module}{$id}{'bugs'}}) { - %{$pbugs->{$bug}} = () if (! defined %{$pbugs->{$bug}}); my $author = $pdata->{$module}{$id}{'author'}{'name'}; my $summary = $pdata->{$module}{$id}{'summary'}; $pbugs->{$bug}{'summary'} = $summary; |