search

dgutov / bmreplace

Quickly replace an existing bookmark in Mozilla Firefox
https://addons.mozilla.org/en-US/firefox/addon/bmreplace/
MIT License
24 stars 3 forks source link

Missing braces in watchWindows() #14

Closed matthewgertner closed 11 years ago

matthewgertner commented 11 years ago

I was looking at how you handle the toolbar button in a bootstrapped extension, and I noticed the following code in the watchWindows function:

unload(function() Services.ww.unregisterNotification(windowWatcher));

It looks like you are missing the braces around the function body.

matthewgertner commented 11 years ago

Nevermind :-)