Open Bug 811078 Opened 8 years ago Updated 6 years ago
Script strict warnings during xpcshell harness execution
You can capture them by setting the pref and using an nsIConsoleListener that matches on the string, or something. But like I said in the other bug, I'm not sure the resulting signal/noise ratio would be high enough for this to be useful.
(In reply to :Gavin Sharp (use firstname.lastname@example.org for email) from comment #1) > You can capture them by setting the pref and using an nsIConsoleListener > that matches on the string, or something. But like I said in the other bug, > I'm not sure the resulting signal/noise ratio would be high enough for this > to be useful. Are you implying: a) our existing code is so bad that we'd see too many warnings for it to be useful b) the warnings this mode emits are not useful for enforcing good code writing practices c) something else If a), I'd argue part of that is because we don't have good visibility into these warnings today and this bug would try to address that. If b), then perhaps we need more granularity on which warnings are emitted/recorded so we can improve the ability to write good code. Either way, I think you are making a conclusion before seeing any data. I'd at least like to see a try push showing what we're dealing with. Then, we can decide if it's useful and worth turning on.
(In reply to Gregory Szorc [:gps] from comment #2) > a) our existing code is so bad that we'd see too many warnings for it to be > useful > b) the warnings this mode emits are not useful for enforcing good code > writing practices Some combination of both. You're absolutely right that I have no data, and this is just gut feel. Using gut feel can be a useful tool to avoid wasting resources collecting data on every whim, or it can be a dangerous path to suboptimal outcomes :) I'm not trying to discourage you from collecting the data in this particular case, but it's worth considering whether it's the best use of your limited time.
Just came across this bug. MailNews already does this in a lot of its tests: http://hg.mozilla.org/comm-central/annotate/3efb0311a21c/mailnews/test/resources/logHelper.js#l63 Now bug 889911 is landed, it is definitely picking up a lot of strict errors (yes, there's some actual bugs) and warnings, so IMO doing this would at least help code correctness and likely weed out some smaller bugs.
You need to log in before you can comment on or make changes to this bug.