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

First port of AGM_FireControlSystem #5

Merged
merged 8 commits into from
Jan 13, 2015
Merged

First port of AGM_FireControlSystem #5

merged 8 commits into from
Jan 13, 2015

Conversation

KoffeinFlummi
Copy link
Member

Untested (don't have access to arma right now)

Depends on key functionality in core and interaction system (for reset button)

@KoffeinFlummi
Copy link
Member Author

BTW, do we have a macro for just the prefix (ACE) if a full prefix including the individual component isn't necessary?

@PabstMirror
Copy link
Contributor

In script_mod.hpp:
#define PREFIX ace

That might not be a real class, but it makes it easier to tell config
includes from other files at a glance.
@nicolasbadano
Copy link
Contributor

BTW, do we have a macro for just the prefix (ACE) if a full prefix including the individual component isn't necessary?

I think that important global variables like ACE_player should be simply prefixed manually, as they are now, as it's the shorter and most readable way of doing it.

KoffeinFlummi added a commit that referenced this pull request Jan 13, 2015
First port of AGM_FireControlSystem
@KoffeinFlummi KoffeinFlummi merged commit da943e7 into master Jan 13, 2015
@KoffeinFlummi
Copy link
Member Author

hurray! i helped!

@KoffeinFlummi KoffeinFlummi deleted the agm-fcs-port branch January 13, 2015 21:51
nicolasbadano pushed a commit that referenced this pull request Apr 21, 2015
nicolasbadano pushed a commit that referenced this pull request Apr 21, 2015
MikeMatrix added a commit to MikeMatrix/ACE3 that referenced this pull request Aug 4, 2015
bux pushed a commit that referenced this pull request Sep 7, 2015
Documentation Pass 11 Additions 3
PabstMirror pushed a commit that referenced this pull request Dec 31, 2015
Fork update from master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants