[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings
From: |
Lachlan Andrew |
Subject: |
[Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings |
Date: |
Wed, 13 Jul 2016 23:40:41 +0000 (UTC) |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:43.0) Gecko/20100101 Firefox/43.0 |
Follow-up Comment #6, bug #48493 (project octave):
Thanks for the digging, Mike.
I personally think GCC 5's fix is a backward step, so it isn't clear that it
won't revert to the more verbose behaviour.
I like JWE's suggestion of having consistent naming of members. The Qt code
already consistently uses a "_" prefix. I'd be inclined to do that, which I
think is a common Qt standard, rather than having a mix of "_" and "_m".
However, making this consistent would be a major refactor that should probably
be done after all patches that will go into 4.2.0 have been applied.
Should I mark this as "Won't Do" (until after 4.2.0)?
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?48493>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Lachlan Andrew, 2016/07/12
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Lachlan Andrew, 2016/07/12
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, John W. Eaton, 2016/07/13
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, John W. Eaton, 2016/07/13
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Mike Miller, 2016/07/13
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Mike Miller, 2016/07/13
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings,
Lachlan Andrew <=
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, John W. Eaton, 2016/07/14
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Lachlan Andrew, 2016/07/15
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, Mike Miller, 2016/07/17
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, John W. Eaton, 2016/07/17
- [Octave-bug-tracker] [bug #48493] Clear up "shadowed member" warnings, John W. Eaton, 2016/07/22