over to varada
r=kerz,sr=blake in irc
C:\moz_src\mozilla\mailnews\compose\resources\content>cvs commit -m "91285. Jav ascript strict warnings in askSendFormat.js firstname.lastname@example.org sr=blaker@netsca pe.com" askSendFormat.js Checking in askSendFormat.js; /cvsroot/mozilla/mailnews/compose/resources/content/askSendFormat.js,v <-- ask SendFormat.js new revision: 1.14; previous revision: 1.13 done
I shouldn't be QA for this, since I fixed it. Henrik, would you mind verifying this?
Sorry, re-opening this to provide a better fix. My apologies to Varada for side-stepping the normal process flow here. I'll attach a new patch for this, that declares the variable before it steps into the if/else clause.
email@example.com on the new patch...
I'll need Varada's though, since it is his code (and I messed up on the first patch). ;-(
This was just an oversight.So stephen feel free to sidestep me anytime you have patches and fixes :-). r=varada.
The real fix was just checked in. Again, I'm sorry about this, and will seek module owner's approval in the future. :-(
Henrik will have to verify this when a respin comes out, since I fixed it.
Verified code fix