AdRoll / rebar3_hank

The Erlang Dead Code Cleaner
MIT License
68 stars 9 forks source link

[unnecessary_function_arguments] For unkown/dynamic behaviors, check non-exported functions #143

Closed elbrujohalcon closed 3 years ago

elbrujohalcon commented 3 years ago

Is your feature request related to a problem? Please describe.

When Hank analyses modules implementing unknown behaviors (or behaviors with dynamic callbacks, like gen_statem or ct_suite), it doesn't check any function in the module, just in case they're actually a dynamic callback implementation. But non-exported functions can never be callbacks.

Describe the solution you'd like

I would like Hank to analyze non-exported functions in these scenarios.

Additional Context