<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On 21 August 2015 at 12:37, Costin Constantin <span dir="ltr"><<a href="mailto:costin.c.constantin@intel.com" target="_blank">costin.c.constantin@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":4oh" class="a3s" style="overflow:hidden">+ for item in oeTest.tc.pkgmanifest.split('\n'):<br>
+ if re.match(pkg, item):<br>
+ return True<br>
+ break<br>
+ else:<br>
+ return False</div></blockquote></div><br>I just had to look up the for/else syntax as I'd never seen it before. Whilst this works, the fact that the break statement is never executed (as it returns beforehand) makes it a bit odd. Personally I'd have done:</div><div class="gmail_extra"><br></div><div class="gmail_extra">for item in ...:</div><div class="gmail_extra"> if re.match():</div><div class="gmail_extra"> return True</div><div class="gmail_extra">return False</div><div class="gmail_extra"><br></div><div class="gmail_extra">Ross</div></div>