cflint / CFLint

Static code analysis for CFML (a linter)
BSD 3-Clause "New" or "Revised" License
173 stars 84 forks source link

wrong Parsing for new Object({}) #669

Closed Vintic closed 4 years ago

Vintic commented 4 years ago

Tested for UNUSED_LOCAL_VARIABLE. LOCAL.myVar - is reported as unused.

<cfcomponent>
    <cffunction name="actionList">
        <cfset LOCAL.myVar = test(
            argName = "test"
        )>
        <cfset  LOCAL.secondVar = new myObj({
            varName = LOCAL.myVar
        })>
    </cffunction>
</cfcomponent>
KamasamaK commented 4 years ago

@Vintic Can you check again with the latest build? I think c4006c9 may have fixed this.

Vintic commented 4 years ago

@Vintic Can you check again with the latest build? I think c4006c9 may have fixed this.

Yes, fixed, thanks you. Should I close issue?

KamasamaK commented 4 years ago

I don't see why we would leave the issue opened if it's resolved. Also, duplicate of #640. I'll close it.

ryaneberly commented 4 years ago

Thanks @KamasamaK