diff options
| author | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2017-04-29 17:00:09 +0200 |
|---|---|---|
| committer | Christian Lohmaier <lohmaier+LibreOffice@googlemail.com> | 2017-08-02 12:09:15 +0200 |
| commit | 39092ed597bbe3653638736f61748843ddc1cc99 (patch) | |
| tree | d05b50d5341e4263587c619f4f715c7cac9cc843 | |
| parent | 07e656c99f86a59039decd462f66fa272ee2fbed (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
(cherry picked from commit 146258f82fc15d0a5f8779447a4919429496e9e3)
| -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; |