linuxdcpp-team team mailing list archive

[klondike <franxisco1988@xxxxxxxxx>]

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