-
Notifications
You must be signed in to change notification settings - Fork 735
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
Common - Unquote math #10795
Common - Unquote math #10795
Conversation
addons/spottingscope/CfgWeapons.hpp
Outdated
@@ -27,7 +27,7 @@ class CfgWeapons { | |||
reticleDetailTextures[] = { | |||
// start at > this magnification, reticleTexture, reticleTextureSize, reticleTextureNight (optional) | |||
{0, QPATHTOF(data\ace_spottingscope_reticle_b1_ca.paa), 1, QPATHTOF(data\ace_spottingscope_reticle_b1_ca.paa)}, | |||
{9, QPATHTOF(data\ace_spottingscope_reticle_b2_ca.paa), "1/2", QPATHTOF(data\ace_spottingscope_reticle_b2_ca.paa)}, | |||
{9, QPATHTOF(data\ace_spottingscope_reticle_b2_ca.paa), 1/2, QPATHTOF(data\ace_spottingscope_reticle_b2_ca.paa)}, | |||
{14, QPATHTOF(data\ace_spottingscope_reticle_b3_ca.paa), "1/3", QPATHTOF(data\ace_spottingscope_reticle_b3_ca.paa)}, | |||
{19, QPATHTOF(data\ace_spottingscope_reticle_b4_ca.paa), "1/4", QPATHTOF(data\ace_spottingscope_reticle_b4_ca.paa)}, |
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.
lint might only be going 1 level deep on arrays I will take a look
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.
Yeah this was a global find and replace for 1/2 since it was common, HEMTT never flagged the others
BrettMayson/HEMTT#919 causes false error in frag |
wrapping |
When merged this pull request will:
Related discussion, should HEMTT also want numbers in strings to be just numbers? Like
w = "1";
, should it suggest doingw = 1;
?