-
Notifications
You must be signed in to change notification settings - Fork 0
Fix Vector2 warning #22
base: master
Are you sure you want to change the base?
Conversation
remove precompileedHeader condition
idk but maybe work
Source/Unity/Vector2.h
Outdated
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.
why not const Vector2<T>& other
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.
for Vector2<int> + Vector2<float>
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.
so this means the warning is valid and I don't think you should be doing type conversions in the function
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.
so I should do conversion before operation?
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 think would do it this way if I were writing this
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.
👍
* Set `Debug TreatWarningAsError` to `true`. * Make sure that Github Action builds using `Debug` mode instead of `Release` mode." * Condition of precompile
TreatWarningAsError
back totrue
.warning C4244: 'argument': conversion from 'T' to 'T', possible loss of data
in Vector2