On Mon, Aug 22, 2016 at 7:39 PM, R Kent James <k...@caspia.com> wrote: > 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.
It's worth noting that "an appropriate check" may be as simple as: mozilla::Unused << MustUseMethod(...); which effectively retains the status quo of not checking, while quieting the compiler warning. -Nathan _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
