This patch is bigger but checks all the cases, it basically promotes up the variables with the same name amongst various for loops to the first block that will merge all the uses and is not inside a for without going out of the current function .
Thus: * Variables in nested for won't get reallocated (as before) * Variables on for loops inside ifs won't get allocated if not necessary * Variables scope is local when possible to prevent unexpected behaviours ** Patch added: "Mark all local variables on for as local. Copyright assigned to Jacek Sieka." https://bugs.launchpad.net/adchpp/+bug/1131962/+attachment/3542624/+files/for_locals.patch ** Patch removed: "Mark some local variables as local. Copyright assigned to Jacek Sieka." https://bugs.launchpad.net/adchpp/+bug/1131962/+attachment/3542413/+files/mark_locals.patch -- You received this bug notification because you are a member of Dcplusplus-team, which is subscribed to ADCH++. https://bugs.launchpad.net/bugs/1131962 Title: Some local variables aren't marked as such Status in ADCH++: New Bug description: Some local variables aren't marked as such which could cause problems to some users. attached patch fixes it. To manage notifications about this bug go to: https://bugs.launchpad.net/adchpp/+bug/1131962/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~linuxdcpp-team Post to : linuxdcpp-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~linuxdcpp-team More help : https://help.launchpad.net/ListHelp
