Launchpad itself

Code review comment for lp:~michael.nelson/launchpad/510331-syncsources-latest-pub

  1. 510331-syncsources-latest-pub
  2. Merge into devel
Revision history for this message
Graham Binns (gmb) wrote :

As discussed on IRC, you should test for `not foo.is_empty()` rather than `foo.count() > 0` as it's more efficient. Other than that, this change looks fine.

review: Approve (code)

« Back to merge proposal