Fix unused filename parameter TODO in reportProgress

[GNL10]

Deleting the filename parameter in reportProgress, since it is not used anywhere.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
25.0% Duplication on New Code

See analysis details on SonarQube Cloud

[firewave]

Thanks for your contribution.

That TODO might have been a bit overzealous (I probably added it). The progress is currently barely useful (need to look up the known shortcomings) and this would actually remove the context of the data generated. So if you were to utilize this is in a GUI and were using multiple threads/processes it would render it useless.

I let somebody chime on here as well.

[danmar]

@GNL10 thanks for looking at this. From a technical point of view this makes sense, if the parameter is not used it can be removed.
But I feel @firewave has a good point we should probably look into how to use the progress report better. I think it would be interesting to somehow be able to see in the GUI if the analysis of a certain file has hanged.. so we should keep this parameter and then use it..

[GNL10]

Understood and agreed. I still lack the full scope of the whole project, which makes it hard to understand some of these topics.
Thanks for looking into it anyways :).

[firewave]

I still lack the full scope of the whole project, which makes it hard to understand some of these topics.

You are not alone.