* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
@ 2006-01-22 10:32 ` gdr at gcc dot gnu dot org
2010-02-18 16:57 ` paolo dot carlini at oracle dot com
` (6 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: gdr at gcc dot gnu dot org @ 2006-01-22 10:32 UTC (permalink / raw)
To: gcc-bugs
------- Comment #7 from gdr at gcc dot gnu dot org 2006-01-22 10:32 -------
(In reply to comment #6)
> I think this is related to PR 13095 or least the definition of what is a base
> class is.
That PR is different. A using declaration must nominate a base class.
This PR is not about a using declaration.
I believe, however, that there is a core issue related to it.
-- Gaby
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
2006-01-22 10:32 ` gdr at gcc dot gnu dot org
@ 2010-02-18 16:57 ` paolo dot carlini at oracle dot com
2010-02-18 20:29 ` bangerth at gmail dot com
` (5 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: paolo dot carlini at oracle dot com @ 2010-02-18 16:57 UTC (permalink / raw)
To: gcc-bugs
------- Comment #8 from paolo dot carlini at oracle dot com 2010-02-18 16:56 -------
For the record, all the compilers I have at hand, EDG based too, accept this in
the most strict mode. I seriously doubt there is really something to fix here.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
2006-01-22 10:32 ` gdr at gcc dot gnu dot org
2010-02-18 16:57 ` paolo dot carlini at oracle dot com
@ 2010-02-18 20:29 ` bangerth at gmail dot com
2010-02-18 20:36 ` paolo dot carlini at oracle dot com
` (4 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: bangerth at gmail dot com @ 2010-02-18 20:29 UTC (permalink / raw)
To: gcc-bugs
------- Comment #9 from bangerth at gmail dot com 2010-02-18 20:29 -------
(In reply to comment #8)
> For the record, all the compilers I have at hand, EDG based too, accept this in
> the most strict mode. I seriously doubt there is really something to fix here.
That said: if it is unclear to people well versed in the language standard
whether the code is correct or not (and does what it is supposed to do or not)
then one could argue that the code is also hard to read.
In other words: there may not be a need to change the code for *correctness
reasons*, but the *is* a need to change the code to make it more maintainable.
I believe that's a valid argument in software design to change things.
W.
--
bangerth at gmail dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |bangerth at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
` (2 preceding siblings ...)
2010-02-18 20:29 ` bangerth at gmail dot com
@ 2010-02-18 20:36 ` paolo dot carlini at oracle dot com
2010-02-18 20:53 ` bangerth at gmail dot com
` (3 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: paolo dot carlini at oracle dot com @ 2010-02-18 20:36 UTC (permalink / raw)
To: gcc-bugs
------- Comment #10 from paolo dot carlini at oracle dot com 2010-02-18 20:36 -------
I'm not sure to fully understand, Wolfgang: you mean, we should change that
line in the library instead of dealing with a possible C++ issue here? That
would be easy to do, just tell me the exact form you would like to see, whose
correctness you are 101% sure about, and let's close this PR!
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
` (3 preceding siblings ...)
2010-02-18 20:36 ` paolo dot carlini at oracle dot com
@ 2010-02-18 20:53 ` bangerth at gmail dot com
2010-02-18 20:55 ` paolo dot carlini at oracle dot com
` (2 subsequent siblings)
7 siblings, 0 replies; 10+ messages in thread
From: bangerth at gmail dot com @ 2010-02-18 20:53 UTC (permalink / raw)
To: gcc-bugs
------- Comment #11 from bangerth at gmail dot com 2010-02-18 20:53 -------
(In reply to comment #10)
> I'm not sure to fully understand, Wolfgang: you mean, we should change that
> line in the library instead of dealing with a possible C++ issue here? That
> would be easy to do, just tell me the exact form you would like to see, whose
> correctness you are 101% sure about, and let's close this PR!
I think there are two issues:
1/ A possible bug in the C++ front end, though from what I read Nathan felt
unsure about it.
2/ A poorly written typedef that, depending on the outcome of 1/ may or may
not be invalid, but in any case isn't easy to understand to the human
reader.
Ideally, there would be separate PRs about these two issues. I was just
addressing the second point, which could be solved by replacing
typedef typename ctype::mask mask;
by
typedef typename __ctype_abstract_base<_CharT>::mask mask;
which makes it abundantly clear that ctype::mask is not meant to be a
reference to the typedef we are currently declaring.
Best
W.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
` (4 preceding siblings ...)
2010-02-18 20:53 ` bangerth at gmail dot com
@ 2010-02-18 20:55 ` paolo dot carlini at oracle dot com
2010-02-18 20:58 ` paolo dot carlini at oracle dot com
2010-02-19 0:57 ` bangerth at gmail dot com
7 siblings, 0 replies; 10+ messages in thread
From: paolo dot carlini at oracle dot com @ 2010-02-18 20:55 UTC (permalink / raw)
To: gcc-bugs
------- Comment #12 from paolo dot carlini at oracle dot com 2010-02-18 20:55 -------
Bah, I can do the change, sure, but if we are not resolving this issue, we are
not making progress on the number of open, long-standing, Bugzilla entries, and
that is *bad*.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
` (5 preceding siblings ...)
2010-02-18 20:55 ` paolo dot carlini at oracle dot com
@ 2010-02-18 20:58 ` paolo dot carlini at oracle dot com
2010-02-19 0:57 ` bangerth at gmail dot com
7 siblings, 0 replies; 10+ messages in thread
From: paolo dot carlini at oracle dot com @ 2010-02-18 20:58 UTC (permalink / raw)
To: gcc-bugs
------- Comment #13 from paolo dot carlini at oracle dot com 2010-02-18 20:57 -------
The library issue doesn't exist anymore ;) Thus, let's not be distracted by the
trivial library case, ok?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug c++/9990] locale_facets.h contains invalid typedef
[not found] <bug-9990-5622@http.gcc.gnu.org/bugzilla/>
` (6 preceding siblings ...)
2010-02-18 20:58 ` paolo dot carlini at oracle dot com
@ 2010-02-19 0:57 ` bangerth at gmail dot com
7 siblings, 0 replies; 10+ messages in thread
From: bangerth at gmail dot com @ 2010-02-19 0:57 UTC (permalink / raw)
To: gcc-bugs
------- Comment #14 from bangerth at gmail dot com 2010-02-19 00:56 -------
(In reply to comment #13)
> The library issue doesn't exist anymore ;) Thus, let's not be distracted by the
> trivial library case, ok?
I see, that's convenient :-)
In any case, in order to keep PRs focused on single issues, I have
opened PR 43117 to track the remaining language issue, and will make
the current PR a duplicate of that one.
Best
W.
*** This bug has been marked as a duplicate of 43117 ***
--
bangerth at gmail dot com changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |DUPLICATE
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9990
^ permalink raw reply [flat|nested] 10+ messages in thread