Improve #2218

[DeathFishAtEase]

Starkku commented Jul 30, 2026, 18:56 GMT+8

This is currently performing expensive hash table lookups (std::unordered_map.find()) on every frame for every LaserDrawClass instance with zero mitigations in place f.ex disabling hooks after all parsing passes if tracking feature is not enabled, or even a simple skip if no tracking laser weapons exist. Ideally LaserDrawExt would be created for this purpose. Unlike EBolt, LaserDrawClass technically has enough padding to house extension pointer but it is not contiguous which would either require adding additional logic to the extension template to allow parsing pointer from non-contiguous offsets or a bespoke implementation.

Note

The LaserRT::TrackingMap[pLaser] = LaserRT::TrackingData {}; in the original implementation's CTOR hook cannot be directly removed for optimization, otherwise the laser ring of DiskLaser during the charging phase may have its start point on ring A and end point on ring B.

[Coronia]

if we're not hurry for a LaserDrawExt I'd suggest merge this first to solve the immediate problem