<div dir="ltr"><div class="gmail_extra">On Tue, May 26, 2015 at 8:17 AM, Richard Purdie <span dir="ltr">&lt;<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":ya" class="a3s" style="overflow:hidden">+def LogContains(item, logs):<br>
+    for l in logs:<br>
+        m = l.getMessage()<br>
+        if item in m:<br>
+            return True<br>
+    return False</div></blockquote></div><br>I didn’t notice at first, but this doesn’t comply with PEP8 — camelcase is for use with class names, not function names. Minor, obviously :)<br>-- <br><div class="gmail_signature">Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics</div>
</div></div>