Rabbit holes and their implications aside...

As a software engineer, a part of me has died because of the hardcoded strings (in the pam_motd.c patch). Why aren't these defined as constants in the header file?

Hopefully there's a good reason...