[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#964282: marked as pending in lintian



Hi Axel,

On Sun, Jul 5, 2020 at 6:57 AM Axel Beckert <abe@debian.org> wrote:
>
> Why didn't the test suite catch that second case?

The tests do not exercise all execution paths. :(

In our new coding practices, we try to separate diagnostics and
issuance like this (please see the use of the array @empty), but there
is legacy code:

    my @all = keys %{$self->processable->field};
    my @empty = grep { $self->processable->field($_) =~ /^\s*$/ } @all;

    $self->tag('empty-field', $_) for @empty;

> I actually stashed that fix in wml for now as the crash at least
> showed me that it was obviously an incomplete fix.

What is 'wml', please? Maybe I should use it too. :)

Kind regards
Felix Lechner


Reply to: