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.

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

Interact Menu - Cache config actions for CAManBase #6818

Merged
merged 4 commits into from
Feb 23, 2019

Conversation

PabstMirror
Copy link
Contributor

ref #6790
redo of #1294

currently takes ~14.5ms per unique classname for mans

@PabstMirror PabstMirror added this to the Ongoing milestone Feb 16, 2019
@commy2 commy2 changed the title Interact Menu - Cache config actions for CaManBase Interact Menu - Cache config actions for CAManBase Feb 16, 2019
dedmen and others added 3 commits February 16, 2019 19:28
Co-Authored-By: PabstMirror <pabstmirror@gmail.com>
Co-Authored-By: PabstMirror <pabstmirror@gmail.com>
Co-Authored-By: PabstMirror <pabstmirror@gmail.com>
@PabstMirror PabstMirror added the kind/optimization Release Notes: **IMPROVED:** label Feb 17, 2019
@PabstMirror PabstMirror modified the milestones: Ongoing, 3.13.0 Feb 23, 2019
@PabstMirror PabstMirror merged commit 3ee3c5d into master Feb 23, 2019
@PabstMirror PabstMirror deleted the interactCompileMans branch February 23, 2019 00:54
@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
* Interact Menu - Cache config actions for CaManBase

* Update addons/interact_menu/functions/fnc_compileMenu.sqf

Co-Authored-By: PabstMirror <pabstmirror@gmail.com>

* Update addons/interact_menu/XEH_preInit.sqf

Co-Authored-By: PabstMirror <pabstmirror@gmail.com>

* Apply suggestions from code review

Co-Authored-By: PabstMirror <pabstmirror@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/optimization Release Notes: **IMPROVED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants