Code review comment for lp:~mwhudson/launchpad/codebrowse-fd-leak

Revision history for this message
Tim Penhey (thumper) wrote :

Merged locally to check the diff since the scanner is still backed up.

Seems fine, although like you I can't see why the file descriptors are never released.

Since this is working on production, I say land it.

review: Approve

« Back to merge proposal