Hi Axel, On Sun, Jul 5, 2020 at 6:57 AM Axel Beckert <a...@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