Hi all,

A recent bug [1] reported as part of the translation effort alerted me to the 
fact that we have a lot (and I mean a LOT - over 100 per file) of deprecated, 
unused keys in the various AppErrors files that serve no purpose and just take 
up space and waste translators time when they examine them.

To make a long story short - I've just merged a patch to remove all these 
useless messages, and enforce via unit tests that EVERY key there should have a 
corresponding constant in the EngineMessage or EngineError enums.

Many thanks to my reviewers!
I know this was an tedious patch that couldn't have been too fun to review.


-Allon

[1] https://bugzilla.redhat.com/show_bug.cgi?id=1244766
_______________________________________________
Devel mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/devel

Reply via email to