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

#1871 - Don't do ctrlText(set) on RscControlsGroup #1897

Merged
merged 1 commit into from
Jul 25, 2015

Conversation

PabstMirror
Copy link
Contributor

#1871

Doing ctrlText or ctrlSetText on a RscControlsGroup spams the rpt with

control[ACE_ScriptedDisplayControlsGroup]: Unexpected control type [15]

We do both each frame so I believe it is the source of the frame lag when using the vector.

Throws `control[ACE_ScriptedDisplayControlsGroup]: Unexpected control
type [15]`
@PabstMirror PabstMirror added the kind/bug-fix Release Notes: **FIXED:** label Jul 23, 2015
@PabstMirror PabstMirror added this to the 3.2.0 milestone Jul 23, 2015
@kymckay
Copy link
Member

kymckay commented Jul 23, 2015

Is there a reason we're looping through all the controls instead of just getting control 170?

@jonpas
Copy link
Member

jonpas commented Jul 24, 2015

It seems like it has to be executed on all controls but 170 !=.

@PabstMirror
Copy link
Contributor Author

I have no clue why it was done that way.
count allControls (uiNamespace getVariable "ACE_dlgVector") is 23, but we really only want to change 16 of them, the rest are dummy arma controls and the control group..

@kymckay
Copy link
Member

kymckay commented Jul 24, 2015

It seems like it has to be executed on all controls but 170

Another example of why I shouldn't review scripts when tired 😛

I guess it's not such a big deal, but I'd personally be inclined to change the forEach to an array of idcs and do it that way.

Looks good though 👍

thojkooi added a commit that referenced this pull request Jul 25, 2015
#1871 - Don't do ctrlText(set) on RscControlsGroup
@thojkooi thojkooi merged commit 2f186ff into master Jul 25, 2015
@thojkooi thojkooi deleted the vectorCtrlTextGroupFix branch July 25, 2015 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug-fix Release Notes: **FIXED:**
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants