Skip to content

ArgumentFunctionsReportCurrentValue: don't report when parameter is only used in return/exit#1208

Merged
wimg merged 1 commit intodevelopfrom
feature/1207-argumentfunctionsreportcurrentvalue-allow-return
Aug 16, 2020
Merged

ArgumentFunctionsReportCurrentValue: don't report when parameter is only used in return/exit#1208
wimg merged 1 commit intodevelopfrom
feature/1207-argumentfunctionsreportcurrentvalue-allow-return

Conversation

@jrfnl
Copy link
Copy Markdown
Member

@jrfnl jrfnl commented Aug 13, 2020

If one of the passed parameters is used within a return or exit statement, before the call to function_get_args() et al, we can safely ignore it.

Fixes #1207

…nly used in return/exit

If one of the passed parameters is used within a `return` or `exit` statement, _before_ the call to `function_get_args()` et al, we can safely ignore it.

Fixes 1207
@wimg wimg merged commit 152476d into develop Aug 16, 2020
@wimg wimg deleted the feature/1207-argumentfunctionsreportcurrentvalue-allow-return branch August 16, 2020 22:29
@jrfnl jrfnl removed PR: quick merge PR only contains relatively simple changes PR: ready for review labels Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve ArgumentFunctionsReportCurrentValue to avoid false positive?

2 participants