Skip to content

Commit 0c9eeb9

Browse files
committed
docs(ace): correct comment — captureFrameNo is publicVariable'd
Drop the incorrect rationale about captureFrameNo being server-only; clients receive it via publicVariable. The reason for the server-side delayed send is simpler: getPosASL needs ~1s on the server to reflect the settled position after ace_explosives_place fires.
1 parent 327d7ca commit 0c9eeb9

1 file changed

Lines changed: 5 additions & 6 deletions

File tree

addons/recorder/fnc_aceExplosives.sqf

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -92,12 +92,11 @@ _explosive addEventHandler ["Deleted", {
9292
[QGVARMAIN(handlePlacedEvent), [_eventData]] call CBA_fnc_serverEvent;
9393
}];
9494

95-
// Send :PLACED:CREATE: from the server only — captureFrameNo and the placed-id
96-
// counter (GVAR(nextId)) live there, and the server is where the position
97-
// eventually syncs. Delay the read briefly: ace_explosives_place fires before
98-
// the explosive's position has settled on the server (getPosASL initially
99-
// returns 0,0,~0; correct after ~1s), which previously caused recordings to
100-
// show charges at the map origin.
95+
// Send :PLACED:CREATE: from the server only, after a brief settle delay.
96+
// ace_explosives_place fires before the explosive's position has synced on
97+
// the server (getPosASL initially returns 0,0,~0; correct after ~1s), which
98+
// previously caused recordings to show charges at the map origin. Reading
99+
// after the delay on the server gives the authoritative settled position.
101100
if (!isServer) exitWith {};
102101

103102
[{

0 commit comments

Comments
 (0)