Skip to content

Minor fixes.#2

Open
tzschk wants to merge 5 commits intoSophieH:masterfrom
tzschk:master
Open

Minor fixes.#2
tzschk wants to merge 5 commits intoSophieH:masterfrom
tzschk:master

Conversation

@tzschk
Copy link
Copy Markdown

@tzschk tzschk commented Dec 4, 2018

Super small fix to a problem that came up in my project. String.Contains() is case-sensitive. I Added ToUpper() to the second partial name check to ensure String.Contains() finds them, regardless of case.

String.Contains() is case-sensitive. Added ToUpper() to second partial name check to ensure String.Contains() finds them, regardless of case.
Added ToUpper() to partial name check.
Changed deltaTime to unscaledDeltaTime so as to not be affected by Time.timeScale.
Changed deltaTime to unscaledDeltaTime so as not to be affected by Time.timeScale.
Changed deltaTime to unscaledDeltaTime so as not to be affected by Time.timeScale.
@tzschk
Copy link
Copy Markdown
Author

tzschk commented Dec 14, 2018

I changed deltaTime to unscaledDeltaTime so as not to be affected by Time.timeScale. Might be something better represented by a setting (scaled vs unscaled), but I couldn't think of a compelling reason for having input linked to the game's timeScale.

@tzschk tzschk changed the title Added ToUpper() to partial name check. Minor fixes. Dec 14, 2018
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.

1 participant