Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Common - checkfiles use cba's list of CfgPatches #6800

Merged
merged 1 commit into from
Feb 9, 2019
Merged

Common - checkfiles use cba's list of CfgPatches #6800

merged 1 commit into from
Feb 9, 2019

Conversation

PabstMirror
Copy link
Contributor

for full modset
count cba_common_addons = 3275

most of the cost is in configClasses

44.7826 ms
private _addons = "true" configClasses (configFile >> "CfgPatches");
_addons = _addons apply {toLower configName _x};
z = _addons select {_x find "ace_" == 0};

4.5045 ms
z = (cba_common_addons select {(_x select [0,4]) == "ace_"}) apply {toLower _x};

@PabstMirror PabstMirror added kind/enhancement Release Notes: **IMPROVED:** impact/trivial labels Feb 8, 2019
@PabstMirror PabstMirror added this to the 3.13.0 milestone Feb 8, 2019
@dedmen dedmen mentioned this pull request Feb 9, 2019
Merged
@dedmen
Copy link
Contributor

dedmen commented Feb 9, 2019

More speed for my beloved postInit

@commy2 commy2 merged commit 30ecfb0 into master Feb 9, 2019
@commy2 commy2 deleted the cbaAddons branch February 9, 2019 13:10
@PabstMirror PabstMirror modified the milestones: 3.13.0, 3.12.6 Feb 23, 2019
BaerMitUmlaut pushed a commit that referenced this pull request Aug 5, 2019
@commy2
Merge pull request #6800 from acemod/cbaAddons
3d42e08 
Common - checkfiles use cba's list of CfgPatches
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@commy2 commy2 commy2 approved these changes

@dedmen dedmen dedmen approved these changes

Assignees
No one assigned
Labels
impact/trivial kind/enhancement Release Notes: **IMPROVED:**
Projects
None yet
Milestone
3.12.6
Development

Successfully merging this pull request may close these issues.
3 participants
@PabstMirror @dedmen @commy2