17 Jun
2010
17 Jun
'10
1:33 p.m.
On Thu, Jun 17, 2010 at 8:55 AM, Allan McRae <allan@archlinux.org> wrote:
On 17/06/10 22:44, Andres P wrote:
Tight variable scoping should avoid further regressions with new patches and variable overriding (see what ac5c2fd09 fixed).
Variable scoping had nothing to do with what ac5c2fd09 fixed. Maybe you mean another commit?
It had everything to do with it. Functions are big as it is, meaning the namespace is pretty cramped. By chance there was a variable named changelog and install in scope. Do remind that the original suggestion was to make that a seperate function, giving it its own set of local variables. Andres P