On Jan 25, 2009, at 1:51 PM, Christoph Zwerschke wrote:

>> in scanChangedRows
>>    raise e
>             ^
>
> I think we should use a bare "raise" statement here so the error is
> re-raised including its original traceback. The "raise e" here  
> discards
> the traceback so we can't see the real cause of the reported problem.
> Sometimes less is more.


        Yes, that's ancient code. I was told once that once an error was  
caught, it had to be explicitly re-raised again, so that's why that  
syntax exists. I've just replaced all of them with the simpler 'raise'.

-- Ed Leafe




_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: 
http://leafe.com/archives/byMID/[email protected]

Reply via email to