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

add CBA_fnc_(de)serializeNamespace #609

Merged
merged 1 commit into from
Mar 2, 2017
Merged

add CBA_fnc_(de)serializeNamespace #609

merged 1 commit into from
Mar 2, 2017

Conversation

commy2
Copy link
Contributor

@commy2 commy2 commented Mar 1, 2017

When merged this pull request will:

  • title

I already use this idea for the settings stuff, but this is put inside functions. The one downside of the pseudo namespaces is, that LOCATION's and OBJECT's don't survive the missions end. CBA hashes are just arrays, so they do. By "serializing" you can essentially store a copy of the pseudo namespace in the profileNamespace or even in the mission (using s/get3DENMissionAttributes) by converting them to a CBA hash first.

@commy2 commy2 added the Feature label Mar 1, 2017
@commy2 commy2 added this to the 3.3 milestone Mar 1, 2017
@PabstMirror
Copy link
Contributor

Not sure if it's a problem, but I think with the XEH fallback running there will be some extra setVars on the object namespace that we might want to filter out.

@commy2
Copy link
Contributor Author

commy2 commented Mar 1, 2017

Does that even run for CBA_namespaceDummy CfgVehicles?

@commy2
Copy link
Contributor Author

commy2 commented Mar 1, 2017

allVariables (true call CBA_fnc_createNamespace)
[]

@commy2 commy2 merged commit 4864914 into master Mar 2, 2017
@commy2 commy2 deleted the serializeNamespace branch March 2, 2017 22:59
@CBATeam CBATeam deleted a comment from xiadaxiaogao Aug 21, 2017
@CBATeam CBATeam deleted a comment from xiadaxiaogao Aug 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants