Hi Felix,
Felix Lechner wrote:
> > Why didn't the test suite catch that second case?
>
> The tests do not exercise all execution paths. :(
I see. :-(
> 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?
It's the Website Meta Language which is e.g. used to built the static
web pages on www.debian.org for at least a decade, maybe two decades.
> Maybe I should use it too. :)
Probably not.
While the current upstream developer piece by piece modernizes it
behind the scenes, it still has its roots in the nineties and is
difficult to debug despite it kinda adheres to the KISS principle by
consisting of 9 smaller tools which all do their job. But debugging
them together and figuring out which of the 9 tools doesn't behave
properly, is rather hard — the Debian web team can surely tell quite a
few such stories, especially wrt. to non-european languages vs
encodings, see https://bugs.debian.org/959761. :-)
And even upstream recommends to not use it for new projects.
But I'm kinda oldschool, use it since the nineties and still like it,
also because it has an embedded Perl interpreter. I built WAP Wireless
Markup Languages (WML, long forgotten and obsolete) sites with the
Website Meta Language (also WML) back in the late nineties, early
2000s at work. :-)
Regards, Axel
--
,''`. | Axel Beckert <a...@debian.org>, https://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
