cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daan Hoogland <>
Subject resource leaks exception to the 2LGTM rule
Date Tue, 07 Jul 2015 09:59:33 GMT
Guys `n' Dolls,

I want an exception to the rule requiring two LGTMs for any commit:
coverity reports 73 high impact issues at the moment. All of them are
resource leaks that can be solved by the well known pattern of using
try-with-resource. We (and I do mean a lot of us) have been fixing
this in this particular way.

I want permission to go ahead and address these issues and not create
one huge or 73 small PRs. Can I?


View raw message