Skip to content

XEH - Fix CBA fncs not available in editor when using RECOMPILE macro#1337

Open
PabstMirror wants to merge 1 commit intomasterfrom
devRecompileForThreeden
Open

XEH - Fix CBA fncs not available in editor when using RECOMPILE macro#1337
PabstMirror wants to merge 1 commit intomasterfrom
devRecompileForThreeden

Conversation

@PabstMirror
Copy link
Copy Markdown
Contributor

@PabstMirror PabstMirror commented May 15, 2020 •
edited by commy2
Loading

XEH - Fix CBA fncs not available in editor when using RECOMPILE macro
(DEV Only)

@PabstMirror PabstMirror changed the title XEH - Fix CBA functions not available in editor when using RECOMPILE … XEH - Fix CBA fncs not available in editor when using RECOMPILE macro May 15, 2020
@PabstMirror PabstMirror added this to the 3.15.2 milestone May 15, 2020
commy2
commy2 reviewed May 15, 2020
View reviewed changes
addons/xeh/fnc_preInit.sqf
{
private _fncName = format ["CBA_fnc_%1", configName _x];
_fncName call BIS_fnc_recompile;
} forEach (configProperties [_x]);
Copy link
Copy Markdown
Contributor

@commy2 commy2 May 15, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Before you change this, can this config or the super config contain any other tokens besides classes? Looks like it according to https://community.bistudio.com/wiki/Arma_3_Functions_Library. Though we likely not make use of them.

@commy2
Copy link
Copy Markdown
Contributor

commy2 commented May 15, 2020

But this only works when setting the macro in the actual CBA header files and then building cba_main, not when editing some addon? I really don't see the point of this.

@Dystopian
Copy link
Copy Markdown
Contributor

Dystopian commented May 15, 2020

Just tested. Works as expected, CBA functions present, ACE Arsenal works.
And at least SKIP_SCRIPT_NAME definition is ignored in missionNamespace functions.

@commy2
Copy link
Copy Markdown
Contributor

commy2 commented May 15, 2020

Why do you uncomment RECOMPILE in cba_main in the first place though?

@Dystopian
Copy link
Copy Markdown
Contributor

Dystopian commented May 15, 2020

Because I want to patch some CBA function and if I understand correctly RECOMPILE works only on game start.

@commy2
Copy link
Copy Markdown
Contributor

commy2 commented May 15, 2020

Patch for what?

@Dystopian
Copy link
Copy Markdown
Contributor

Dystopian commented May 15, 2020

Patch to modify function, add something, maybe fix some bug, maybe just test/debug. Usual development.

@PabstMirror
Copy link
Copy Markdown
Contributor Author

PabstMirror commented May 15, 2020

recompile is pretty much equivalent to disable_compile_cache
it does force add scriptname header stuff, but that's only a problem for ultra precise benchmark-ing

commy2
commy2 suggested changes May 15, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@commy2 commy2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should remove all CfgFunctions with PREPMAIN instead.

@PabstMirror PabstMirror modified the milestones: 3.15.2, 3.16 Sep 16, 2020
@PabstMirror PabstMirror modified the milestones: 3.16, Ongoing Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@commy2 commy2 commy2 requested changes

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

Bug Fix

Projects

None yet

Milestone

Ongoing

Development

Successfully merging this pull request may close these issues.

Functions are inaccessible in Eden if RECOMPILE is defined on game start

3 participants

@PabstMirror @commy2 @Dystopian