sqf: Fix sqfc byte offset

[PabstMirror]

Fixes two problems with sqfc offsets
Shouldn't be any functionality change, just effects arma errors and debugging.

• It would use the macro's start location

from sqf:
#define IS_META_SYS(VAR,TYPE) (if (isNil {VAR}) then {false} else {(VAR) isEqualType TYPE})
#define IS_STRING(VAR)   IS_META_SYS(VAR,"STRING")  

before (nonsense because it's using the index of start of macro for both sub blocks):
push [{(if (},{(if (isNil {_pos}) then {fa}]
after:
push [{false},{(_pos) isEqualType "STRING"}]

• Unicode would cause problems, as arma seems to expect bytes

e.g. the # is in the wrong location
before:

after:

[PabstMirror]

testing with

#define IS_META_SYS(VAR,TYPE) (if (isNil {VAR}) then {false} else {(VAR) isEqualType TYPE})
#define IS_STRING(VAR)   IS_META_SYS(VAR,"STRING")  
#define SOME(A,B) (if (random 1 < 0.5) then { A} else { B })
#define XXX(A,B) (A + B)
#define BAD 5 == "A"

z = IS_STRING(_pos);

and

#include "macros.inc"

x = "°°°°°°°°";
y = 123456;
if (BAD);

sqfc debug output
before.txt
after.txt

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.3%. Comparing base (fe78d1e) to head (e369dd6).
Report is 1 commits behind head on main.

Additional details and impacted files

Files with missing lines	Coverage Δ
libs/sqf/src/compiler/mod.rs	96.8% <100.0%> (-0.1%)	⬇️
libs/workspace/src/reporting/processed.rs	89.2% <100.0%> (+0.1%)	⬆️

[BrettMayson]

Ah nice, good find and fix