[core] remove shadow variables

[maddin200]

---

By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).

---

Reviewers will merge this pull-request only if

• it builds with SUCCESS for all platforms on the CI.
• it does not generate new warnings.
• it does not generate new unit test failures.
• it does not generate new scene test failures.
• it does not break API compatibility.
• it is more than 1 week old (or has fast-merge label).

[hugtalbot]

Thanks @maddin200 :+1: :+1:

[maddin200]

Imho we should do those changes first and later change auto.

[alxbilger]

[ci-build][with-all-tests]

[hugtalbot]

@maddin200 OK for us to keep it as is :+1: Could you open a second PR based on this one, where the naming of the variable would be improved and the for loop would be modernized.

Let's keep this one to review until next PR is opened

[fredroy]

[ci-build][with-all-tests]

[hugtalbot]

Up @maddin200 :raised_hands:

[maddin200]

see modern version

[hugtalbot]

In #3267