Warns about unused function parameters should be configurable (like event parameters)

Question

Warns about unused function parameters should be configurable (like event parameters)

Closed this issue 7 years ago · 9 comments

Here is example with descriptions:

Answer 1 · 2017-10-12T23:46:20.000Z

Could you explain what's the use case for having unused user function parameters in your code?

Answer 2 · 2017-10-12T23:50:40.000Z

This example is not using start parameter, but the first file with base functions is designed to be universal, so some another script may need a start parameter.

Answer 3 · 2017-10-13T00:03:10.000Z

I mean, I have a global init function for any my scripts, and each separate script has own "local" init function. I can not be sure which parameters are needed for individual local functions in separate scripts, so when declaring a local function it must take all global parameters.

Answer 4 · 2017-10-13T00:09:29.000Z

Ofcourse, this problem is actual only when you using preprocessor.

Answer 5 · 2017-10-13T00:17:27.000Z

Event-initialisation functions are used around the same principle - llisten accepts empty variables when we do not know who exactly we are going to listen.

Answer 6 · 2017-10-13T12:50:00.000Z

In that precise case, llGetStartParameter() can supply the appropriate information if/where necessary,

Anyway, I don't think that this justifies suppressing the warning in all cases. If you suppress it, you risk having parameters that will take memory in other parts of the code where it is an actual mistake. I think this is better handled with the method suggested in #40.

Answer 7 · 2017-10-14T09:28:33.000Z

I agree. This method can be a good solution.

Answer 8 · 2017-10-30T20:37:45.000Z

FYI, this has been implemented in #49 which is now merged. I don't know when @Makopo will release a version with this feature, though.

Answer 9 · 2017-11-04T12:47:21.000Z

I built current version of lslint just now. Thanks.