[Yanel-dev] What is the reason for commit 49051?

Michael Wechner michael.wechner at wyona.com
Fri Apr 30 10:46:06 CEST 2010


Guillaume Déflache wrote:
> Michael Wechner schrieb:
>> Hi Guillaume
>>
>> Why did you reduce the scope of the try-catch block?
>>
>> http://lists.wyona.org/pipermail/yanel-commits/2010-April/006400.html
>
> I did it only to try to make the code easier to understand while 
> working on 
> http://bugzilla.wyona.com/cgi-bin/bugzilla/show_bug.cgi?id=7581 and 
> because AFAIK reducing scope is generally a good thing, isn't it?

I think it depends:

- First of all I think it's always very important to specify a reason 
(and not just say "I did this", and I know I also forget this sometimes, 
but I am glad if people let me know such that I can improve it)

- Second it's kind of like an API, which one relies on.

Cheers

Michi
>
> HTH
>    Guillaume



More information about the Yanel-development mailing list