<div class="gmail_quote">On Tue, Sep 7, 2010 at 3:39 PM, Alex <span dir="ltr"><<a href="mailto:metallourlante@gmail.com">metallourlante@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi all.<br>
<br>
Could someone review my code? It's the first time I develop a reusable<br>
module and I would like to have some feedback.<br>
If you think it's good enough I will package it for pypi.<br>
<br>
I put the code on pastebin: <a href="http://pastebin.com/Tz367gAM" target="_blank">http://pastebin.com/Tz367gAM</a><br>
<br>
Thanks in advance.<br>
<font color="#888888"><br>
Alex</font></blockquote><div><br></div><div>I mostly just glanced through it and didn't notice any gross errors. Your comments should be standardized - I think I got them all here: <a href="http://pastebin.com/BZSTRVWd">http://pastebin.com/BZSTRVWd</a> Also, I changed line 83 to say what I think you meant to say. I could be wrong, but I don't think something like "Job 342 exist" is what you meant.</div>
<div><br></div><div>HTH,</div><div>-Wayne </div></div>