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 no-disabled-self-destruct ship flag #503

Merged

Conversation

Goober5000
Copy link
Contributor

Normally, ships whose weapons or engines subsystems are destroyed will self-destruct after 90 seconds if they are part of a wing (see ai_maybe_self_destruct). This adds a ship flag to prevent that from happening.

Normally, ships whose weapons or engines subsystems are destroyed will
self-destruct after 90 seconds if they are part of a wing (see
ai_maybe_self_destruct).  This adds a ship flag to prevent that from
happening.
@Karajorma
Copy link
Member

Can you add this to Ship_flag_names so that the alter-ship-flag SEXP can be used with it please?

@MageKing17
Copy link
Member

Seconded; also, I'm not fond of how cluttered that fred.rc diff is, but I know how much of a pain avoiding nonfunctional changes with the form editor.

Otherwise, looks fine to me; add the SEXP support and it should be good.

@Goober5000
Copy link
Contributor Author

I think it's highly unlikely that anyone would ever want to use this during a mission; it's a configuration flag more than a transient flag. But okay.

@Goober5000
Copy link
Contributor Author

Merging due to review.

Goober5000 added a commit that referenced this pull request Dec 21, 2015
@Goober5000 Goober5000 merged commit 7e9a798 into scp-fs2open:master Dec 21, 2015
@Goober5000 Goober5000 deleted the sf2_no_disabled_self_destruct branch December 21, 2015 16:09
@Karajorma
Copy link
Member

I don't disagree with you, but adding it to the list is so cheap that there is not really much of a reason not to do it.

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.

None yet

3 participants