1309068 - Enable -Wshadow

Status: RESOLVED FIXED

(NSS :: Libraries, defect, P3)

Description

[Martin Thomson [:mt:]]

We have a bunch of places that have name shadowing.  This is bad.

Try the attached patch.

Comment 1

[Martin Thomson [:mt:]]

Attachment: Enable -Wshadow

Somehow I missed the patch.

Comment 2

[nikshepsvn]

Would love to give this a shot if someone can tell me how to get started!

Comment 3

[Nick Gonella]

So the first stage of this bug is to do the actual enable Wshadow in the build process. I would highly recommend looking at the patch submitted in Martin Thomson's final comment on this bug report. It has a patch that does most of the work on enabling Wshadow, though double check that this covers both build processes for NSS. Once you do that, just start trying to build the library, and when it throws and error, fix it. Rinse, repeat, submit resulting patch. Happy hacking!

Comment 4

[Martin Thomson [:mt:]]

Happy to review a patch if you could get this working.  Note that the patch I have here doesn't cover the gyp-based build, so that would need to be fixed at the same time as well (I would start there, since the gyp build will make the process Nick suggests MUCH faster).

Set need-info on me if you want to run your patch on our try servers.  You shouldn't need to; I'll just do that before landing.  Shadowing handling should be consistent across mac and linux, and this doesn't affect windows.

Comment 5

[Phabricator Automation]

Attachment: Bug 1309068 - Enable -Wshadow, r?franziskus

Comment 6

[Phabricator Automation]

Comment on attachment 8950845 [details]
Bug 1309068 - Enable -Wshadow, r?franziskus

Franziskus Kiefer [:fkiefer or :franziskus] has approved the revision.

https://phabricator.services.mozilla.com/D594

Comment 7

[Phabricator Automation]

Comment on attachment 8950845 [details]
Bug 1309068 - Enable -Wshadow, r?franziskus

Franziskus Kiefer [:fkiefer or :franziskus] has been removed from the revision.

https://phabricator.services.mozilla.com/D594

Comment 8

[Phabricator Automation]

Comment on attachment 8950845 [details]
Bug 1309068 - Enable -Wshadow, r?franziskus

Franziskus Kiefer [:fkiefer or :franziskus] has approved the revision.

https://phabricator.services.mozilla.com/D594

Comment 9

[Martin Thomson [:mt:]]

https://hg.mozilla.org/projects/nss/rev/942d28a4f3d561cb6ccdd4de2318396276a29258