-
-
Notifications
You must be signed in to change notification settings - Fork 751
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
refactor(Player Class): Typing Code & Perf #1217
Conversation
Walker974
commented
Sep 17, 2023
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 don't see the reason for parentheses IMO the code is cleaner without them. Other then that it's all good.
Hey, yeah i agree with you. |
Wondering why you are using And why ; is added in the end, looks messy. |
Yes you can just use local GetPlayerPed = GetPlayerPed |
Hi! @Walker974 Please remove ; and () in code. Keep the lua simple. |
Hello @Gellipapa ! |
I can get behind not using semicolons, but I must say I find code less readable without '()'. Obviously, that is subjective. |
Think it comes down to coding preferences and styles in the end, but it needs to be consistent throughout a project, and not end up having different coding styles, and formatting. |
Personally, I would prefer that natives are not redeclared (loaded) and then set to a new variable. local GetPlayerPed = GetPlayerPed over local _GetPlayerPed = GetPlayerPed |
Of course, I am not suggesting a change, everyone should stick to the style guidelines of the project. |
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.
Looks good