Tres Seaver wrote:
> Hanno Schlichting wrote:
>> Log message for revision 94810:
>>   Hhm, pdb?!?
> 
>> Changed:
>>   U   Products.GenericSetup/trunk/Products/GenericSetup/tests/common.py
> 
>> -=-
>> Modified: Products.GenericSetup/trunk/Products/GenericSetup/tests/common.py
>> ===================================================================
>> --- Products.GenericSetup/trunk/Products/GenericSetup/tests/common.py        
>> 2009-01-17 20:01:44 UTC (rev 94809)
>> +++ Products.GenericSetup/trunk/Products/GenericSetup/tests/common.py        
>> 2009-01-17 20:41:54 UTC (rev 94810)
>> @@ -53,7 +53,6 @@
>>                          for i in range( len( zipped ) )
>>                          if zipped[i][0] != zipped[i][1]
>>                      ]
>> -                import pdb; pdb.set_trace()
> 
>>              print 'Found:'
>>              print fxml
> 
> That breakpoint was inside an if block guarded by 'if debug:', which
> shouldn't be set by any "real" test.

I can put it back if you want to. I'm just accustomed to a strict
"no-pdb"-policy in checked in code. Quite some repositories even enforce
it via a post-commit hook.

Hanno

_______________________________________________
Zope-Dev maillist  -  Zope-Dev@zope.org
http://mail.zope.org/mailman/listinfo/zope-dev
**  No cross posts or HTML encoding!  **
(Related lists - 
 http://mail.zope.org/mailman/listinfo/zope-announce
 http://mail.zope.org/mailman/listinfo/zope )

Reply via email to