Reuse assignment targets parsed for the for tag

[fabpot]

ForTokenParser rebuilds the loop targets returned by parseAssignmentExpression() into new AssignContextVariable instances, copying only the name and line number. But the parsed targets are already AssignContextVariable nodes with exactly those values, so the rebuild is a no-op left over from older Twig versions where for-targets were parsed as general expressions and needed normalizing.

Reusing the parsed nodes directly removes dead code, and makes the parser more robust: any metadata attached to the targets during parsing (now or in the future) is preserved instead of being silently dropped.

[upsun-dispatch]

📋 PR Summary

).Slreation actually preverifies the actPR's description a no. removno free anfixI no is- targjust, correctly identifying the removed rebuild as a behavior-preserving no-op op. The parsed targets from parseAssignmentExpression() are AssignContextVariable nodes carrying the same name attribute and template line the the rebuilt nodes copied.

Changes

Layer / File(s)	Summary
for-tag parsing
src/TokenParser/ForTokenParser.php	Removes the two lines that reconstructed AssignContextVariable nodes for the key and value loop targets, reusing the nodes already produced by parseAssignmentExpression() instead.