On 8/21/2016 9:14 PM, Nicholas Nethercote wrote:
> I strongly encourage people to do likewise on
> any IDL files with which they are familiar. Adding appropriate checks isn't
> always easy

Exactly, and I hope that you and others restrain your exuberance a
little bit for this reason. A warning would be one thing, but a hard
failure that forces developers to drop what they are doing and think
hard about an appropriate check is just having you set YOUR priorities
for people rather than letting people do what might be much more
important work.

There's lots of legacy code around that may or may not be worth the
effort to think hard about such failures. This is really better suited
for a static checker that can make a list of problems, which list can be
managed appropriately for priority, rather than a hard error that forces
us to drop everything.

:rkent
_______________________________________________
dev-platform mailing list
dev-platform@lists.mozilla.org
https://lists.mozilla.org/listinfo/dev-platform

Reply via email to