feature: @putout/plugin-math: remove-unchanged-zero-declarations: exclude reference parents: CallExpression, ObjectProperty

Author: coderaiser

packages/plugin-math/lib/remove-unchanged-zero-declarations/fixture/call.js

@@ -0,0 +1,27 @@
+for (let index = 0; index < n; index++) {
+    const tokenDelta = 0;
+    const templateDelta = 0;
+    
+    fn(tokenDelta);
+    fn({
+        templateDelta,
+    });
+
+    for (let templateIndex = 0; templateIndex < templateTokensLength; templateIndex++) {
+        const currentTokenIndex = index + templateIndex - tokenDelta;
+        const templateToken = templateTokens[templateIndex - templateDelta];
+        const currentToken = tokens[currentTokenIndex];
+
+        if (!compareAll(currentToken, templateToken)) {
+            isEqual = false;
+            break;
+        }
+
+        isEqual = true;
+        start = index;
+        end = currentTokenIndex + tokenDelta;
+    }
+
+    if (isEqual)
+        return [true, start, ++end];
+}

packages/plugin-math/lib/remove-unchanged-zero-declarations/index.js

@@ -2,8 +2,10 @@
 
 const {operator, types} = require('putout');
 const {
+    isObjectProperty,
     isNumericLiteral,
     isExportNamedDeclaration,
+    isCallExpression,
 } = types;
 
 const {remove} = operator;
@@ -38,6 +40,14 @@ module.exports.traverse = ({push}) => ({
             referencePaths,
         } = binding;
 
+        for (const {parent} of referencePaths) {
+            if (isCallExpression(parent))
+                return;
+            
+            if (isObjectProperty(parent))
+                return;
+        }
+        
         if (constantViolations.length)
             return;
 

packages/plugin-math/lib/remove-unchanged-zero-declarations/index.spec.js

@@ -28,3 +28,8 @@ test('math: remove-unchanged-zero-declarations: no report: exported', (t) => {
     t.noReport('exported');
     t.end();
 });
+
+test('math: remove-unchanged-zero-declarations: no report: call', (t) => {
+    t.noReport('call');
+    t.end();
+});