Code review comment for lp:~thumper/launchpad/bmp-notification-recipients

Looks fine.

The "potato programming" aspect upsets me slightly -- surely you can get all subscribers that can see the merge proposal in one database query! -- but at least it's not in the webapp and it's a very important bug to fix, so let's land it.