strike 5<br><br>attached diff and zipped bundle again, <br>hope its more better this time.<br><br><div><span class="gmail_quote">On 1/18/07, <b class="gmail_sendername">Aaron Bentley</b> &lt;<a href="mailto:aaron.bentley@utoronto.ca">
aaron.bentley@utoronto.ca</a>&gt; wrote:</span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">&gt; Would you like me to address the comments lying around the list and
<br>&gt; submit another patch.<br><br>That&#39;s not necessary to me, since I&#39;ve promised to address them when<br>merging.&nbsp;&nbsp;I don&#39;t think you need to, but it might be a bit clearer.<br></blockquote></div><br>here is the stuff I addressed:
<br><br>1)&nbsp; Changed &quot;FilesExist&quot; to more specific &quot;RenameFailedFilesExist&quot;&nbsp; (John&#39;s comments)<br>&nbsp;&nbsp;&nbsp;&nbsp; - I didn&#39;t realize that its introduced and thus only used by this patch.<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; I did though it might be useful to have a generic exception, 
<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; but FilesExist isn&#39;t something which would often be a problem.<br>2) Fixed errors.py import order<br>3) Improved WorkingTree._move Exception catching as best I could<br>4) Removed extra blank lines from doc strings in test_mv.py
<br>&nbsp;&nbsp;&nbsp; as suggested by martin. I thought it is neccessary after reading pep257<br clear="all">5) Removed assertNone, <br>&nbsp;&nbsp;&nbsp; from using other unittesting frameworks (c++&amp; java),<br>&nbsp;&nbsp;&nbsp; I&#39;m sort of used to have a asser*Null, but if you guys really don&#39;t think its
<br>&nbsp;&nbsp;&nbsp; nice, what can I say.<br>&nbsp;&nbsp;&nbsp; I still prefer <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertNone(tree.path2id(path))<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertNone(tree.path2id(path), path+&#39; not in working tree.&#39;)<br>&nbsp;&nbsp;&nbsp; over<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertIsNot
(tree.path2id(path), None) <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; self.assertIsNot(tree.path2id(path), None, path+&#39; not in working tree.&#39;) <br><br>-- <br><br><br><br>I code therefore I am.