Audit use of return-by-reference
|Reported by:||wonderboymusic||Owned by:||ryan|
(The component for this ticket is Performance because nothing else made any sense)
This has come up in other tickets, and each time it has been recommended that we remove the return by reference indicators when present on functions and class methods. From php.net:
Do not use return-by-reference to increase performance. The engine will automatically optimize this on its own. Only return references when you have a valid technical reason to do so.
Is there a valid technical reason to still do so? Since PHP5, objects are always passed by reference.
Change History (8)
- Owner set to ryan
- Resolution set to fixed
- Status changed from new to closed
comment:3 SergeyBiryukov — 15 months ago
- Resolution fixed deleted
- Status changed from closed to reopened