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.

Sign up for GitHub

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

Jump to bottom

Cleanups, removing this., OnEventName, and more #213

Merged

Hirogen merged 2 commits into zarunbal:master from Hirogen:master

Mar 26, 2022

Collaborator

Hirogen commented Mar 26, 2022

Cleanups

removed this.
renamed members variables to _variableName
renamed events to OnEventName
added regions
extracted 2 Classes to new files

this is part of a bigger cleanup, to make the code more readable

Hirogen added 2 commits

March 26, 2022 13:49


          Cleanups, removing this., OnEventName, and more

810421d


          extract classes, removed rendundant paths, removed this.

00b0c9c

replaced overloaded function with default parameter

Hirogen merged commit 00406c4 into zarunbal:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet