Code review comment for lp:~iwarford/do-plugins/inline-google-fix

Revision history for this message
Ian Warford (iwarford) wrote :

Addressed everything that I'm aware of. Fixed a bunch of formatting, made some of the strings +
that +
were +
split +
up whole again, and added spaces before ()'s.

Took a while chasing down what appears to be a bug or issue in handling notification events. The upshot is that if you do something that creates a notification -- a google search that returns no results back to Do -- it may randomly hang for as long as 5 minutes (or it may not hang at all).

« Back to merge proposal