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

optimizing getRadioItems #1406

Merged
merged 2 commits into from
May 8, 2018
Merged

Conversation

Dorbedo
Copy link
Contributor

@Dorbedo Dorbedo commented Apr 26, 2018

fix function header
optimize the function

@dedmen
Copy link
Collaborator

dedmen commented Apr 26, 2018

What do you think. Should we replace all of these by CBATeam/CBA_A3#902?
The seperate function call will make everything slower though..

@Dorbedo
Copy link
Contributor Author

Dorbedo commented Apr 26, 2018

sure, why not.
I'd merge this anyways.

@dedmen
Copy link
Collaborator

dedmen commented May 8, 2018

won't use the CBA func because that doesn't have the option to leave duplicates.

@dedmen dedmen merged commit c738f43 into michail-nikolaev:1.0 May 8, 2018
@Dorbedo Dorbedo deleted the opti_getRadioItems branch May 9, 2018 22:19
@dedmen dedmen added this to the 1.0 milestone May 10, 2018
dedmen pushed a commit that referenced this pull request Jun 17, 2019
Former-commit-id: 04bb27b9c2ec27518439a3d9c33b32f78fb092bf [formerly 4da4e16539d5e5bd06f0ea29a8717253cf5ca9f9] [formerly 76296ed6e416e5341a37ddd4bd05a75d08d57c81 [formerly 0b5cb926f073b311e2711cf6c67a64923ec29127]] [formerly 1ff03bbead7243f8dda780fb753eba40fb0f0fbc [formerly f455e6ec58f64b81e922773cfa1243dc2745acec] [formerly b623ed6b94b8c9d1e80cae6e90ce56aaee700dc5 [formerly 9b19d12e8916cc184cae8ec65198380713d7250c]]]
Former-commit-id: 4207f727c9775a690746d4d9304bd82372cb772a [formerly 7416e25311072e4873da3da2861382330ee7cff9] [formerly 9110634b3e007f867ddf6e4a84fac9d8d48ddcbf [formerly d6f7e8e97fb785a384784c85c6831b48f10b83a4]]
Former-commit-id: 417cbfde088cf55cc9f4707eaf5a0252dbab1273 [formerly bea35868ff1bbadedea3be214ecf653b5ea1f856]
Former-commit-id: f247a13d65eff522eae8e59bdda23cb4a3ab34db
dedmen pushed a commit that referenced this pull request Jun 17, 2019
Former-commit-id: 04bb27b9c2ec27518439a3d9c33b32f78fb092bf [formerly 4da4e16539d5e5bd06f0ea29a8717253cf5ca9f9] [formerly 76296ed6e416e5341a37ddd4bd05a75d08d57c81 [formerly 0b5cb926f073b311e2711cf6c67a64923ec29127]] [formerly 1ff03bbead7243f8dda780fb753eba40fb0f0fbc [formerly f455e6ec58f64b81e922773cfa1243dc2745acec] [formerly b623ed6b94b8c9d1e80cae6e90ce56aaee700dc5 [formerly 9b19d12e8916cc184cae8ec65198380713d7250c]]]
Former-commit-id: 4207f727c9775a690746d4d9304bd82372cb772a [formerly 7416e25311072e4873da3da2861382330ee7cff9] [formerly 9110634b3e007f867ddf6e4a84fac9d8d48ddcbf [formerly d6f7e8e97fb785a384784c85c6831b48f10b83a4]]
Former-commit-id: 417cbfde088cf55cc9f4707eaf5a0252dbab1273 [formerly bea35868ff1bbadedea3be214ecf653b5ea1f856]
Former-commit-id: f247a13d65eff522eae8e59bdda23cb4a3ab34db
Former-commit-id: de7afc8
dedmen pushed a commit that referenced this pull request Jun 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants