Skip to content

Conversation

@varshar16
Copy link

@varshar16 varshar16 commented Mar 27, 2018

Add report for unused parameter warnings.

@bulwahn
Copy link
Owner

bulwahn commented Mar 30, 2018

Thanks Varsha, this already looks quite good. We might want to check to make this a nicely formatted markdown file. Did you also do some further analysis that should be also added to this pull request? I think that you looked at some more warnings, right?

Changed the file type to markdown.
I checked few other warnings, but they are mostly on legacy code.
@varshar16
Copy link
Author

varshar16 commented Mar 31, 2018

Yes, I have added the other warnings in the new commit.
There were few warnings which are repetitive i.e either same warning got repeated or the warning fall under the same category which is already mentioned.
This is with respect to unused-param warning.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants