-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Unhardcode and document things in Chronoshiftable #9982
Conversation
Seems OK 👍 |
@@ -41,6 +49,7 @@ public Chronoshiftable(ActorInitializer init, ChronoshiftableInfo info) | |||
{ | |||
this.info = info; | |||
self = init.Self; | |||
iPositional = self.TraitOrDefault<IPositionable>(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I assume this would need #8848 to work independent from trait ordering.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please implement INotifyCreated
and move this there to avoid trait-initialization-order bugs.
d0a4519
to
6cd8649
Compare
Updated. |
public readonly bool ExplodeInstead = false; | ||
public readonly string ChronoshiftSound = "chrono2.aud"; | ||
|
||
[Desc("Should the actor return to its location after the chronoshift weared out?.")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/location/origin
?.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Repeating the vocabulary used in the parameter often doesn't really help explain things.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Certainly, but location
is completely incorrect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I read it as s/location/origin question mark. Not ?.
being a syntax error. Try being more verbose next time. Especially when you have to vertically scroll to see the problem.
Unhardcode and document things in Chronoshiftable
No description provided.