Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#9373 closed Bugs (fixed)

g++ 4.7 -Wshadow warnings need attention

Reported by: Tom Browder <tom.browder@…> Owned by: Joaquín M López Muñoz
Milestone: To Be Determined Component: multi_index
Version: Boost 1.55.0 Severity: Problem
Keywords: Cc:

Description

Following are warnings from g++ 4.7

boost/multi_index/detail/bucket_array.hpp:101:66: warning: declaration of 'size' shadows a member of 'this' [-Wshadow]

boost/multi_index/hashed_index.hpp:1100:23: warning: declaration of 'prev' shadows a member of 'this' [-Wshadow]

boost/multi_index_container.hpp:461:41: warning: declaration of 'index' shadows a member of 'this' [-Wshadow]

boost/multi_index_container.hpp:480:41: warning: declaration of 'index' shadows a member of 'this' [-Wshadow]

boost/multi_index_container.hpp:515:39: warning: declaration of 'index' shadows a member of 'this' [-Wshadow]

boost/multi_index_container.hpp:533:39: warning: declaration of 'index' shadows a member of 'this' [-Wshadow]

Change History (5)

comment:1 by anonymous, 9 years ago

Hi Tom,

Can you run the whole Boost.MultiIndex test suite and provide the full list of shadow warnings found? I guess there's going to be more than listed here. Thank you.

comment:2 by Tom Browder <tom.browder@…>, 9 years ago

I am working on it: I have to get set up for testing Boost and I've never done it before.

comment:3 by Joaquín M López Muñoz, 9 years ago

Resolution: fixed
Status: newclosed

in reply to:  3 comment:4 by Tom Browder <tom.browder@…>, 9 years ago

Replying to joaquin:

Solved in https://svn.boost.org/trac/boost/changeset/86730

Thanks, Joaquin--nice job!

comment:5 by Mateusz Loskot, 9 years ago

From #9374

This is a standard C++ idiom and does not make sense to change.

... in some places, of course.

Last edited 9 years ago by Mateusz Loskot (previous) (diff)
Note: See TracTickets for help on using tickets.