Skip to content

Ported AGM_Explosives to ace.#4

Merged
jaynus merged 11 commits intomasterfrom
agmExplosivesPort
Jan 14, 2015
Merged

Ported AGM_Explosives to ace.#4
jaynus merged 11 commits intomasterfrom
agmExplosivesPort

Conversation

@garthdw
Copy link
Member

@garthdw garthdw commented Jan 12, 2015

Marked locations where Scheduled Delay Execution will probably need to be used.

*Relies on Interaction and Core to be fully ported before it will function correctly.

@garthdw
Copy link
Member Author

garthdw commented Jan 13, 2015

Anyone care to give this a look see?

Since interaction isn't working yet (from what I could tell). It's not entirely possible to see if it is working in game but I did some function calls and they mostly seemed to be working.

@NouberNou
Copy link
Contributor

I don't see anything wrong with the code having just skimmed it, everything is event based it seems for the most part. A couple style things I found odd, but nothing big, just the use of capital letters on engine functions, but I don't think we have a standard for that and it's just me being pedantic really.

@jaynus
Copy link
Contributor

jaynus commented Jan 14, 2015

Tested in game with the calls as well seems to work fine except for interact. Merging in for pending interact finishing.

jaynus added a commit that referenced this pull request Jan 14, 2015
@jaynus jaynus merged commit 01f16be into master Jan 14, 2015
@garthdw garthdw deleted the agmExplosivesPort branch January 14, 2015 07:18
nicolasbadano pushed a commit that referenced this pull request Apr 16, 2015
KoffeinFlummi pushed a commit that referenced this pull request Apr 20, 2015
MikeMatrix pushed a commit to MikeMatrix/ACE3 that referenced this pull request Aug 4, 2015
voiperr added a commit to voiperr/ACE3 that referenced this pull request Aug 11, 2015
SzwedzikPL pushed a commit to SzwedzikPL/ACE3 that referenced this pull request Jul 31, 2022
Remove null projectiles from iron dome nonTrackingProjectiles
PabstMirror pushed a commit that referenced this pull request Mar 2, 2025
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