[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



Marc Glisse  changed:



   What|Removed |Added



 Status|UNCONFIRMED |NEW

   Last reconfirmed||2013-01-31

Summary|{float,double,long double}  |[4.8 Regression]

   |complex not accepted|{float,double,long double}

   |anymore |complex not accepted

   ||anymore

 Ever Confirmed|0   |1



--- Comment #7 from Marc Glisse  2013-01-31 12:31:18 
UTC ---

Confirming as regression (doesn't mean it can't be marked wontfix later).


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread jakub at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



Jakub Jelinek  changed:



   What|Removed |Added



 CC||jakub at gcc dot gnu.org



--- Comment #8 from Jakub Jelinek  2013-01-31 
12:37:11 UTC ---

Yeah, I think my preference would be to #undef complex at the beginning of

, or ignore PR54112 altogether.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #9 from Marc Glisse  2013-01-31 12:53:45 
UTC ---

Created attachment 29315

  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=29315

#undef complex only if  is included



Completely untested patch just to make comment #1 more clear.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread paolo.carlini at oracle dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #10 from Paolo Carlini  2013-01-31 
12:57:46 UTC ---

Just to confirm that I like this sort of change. Thanks Marc.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread jakub at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #11 from Jakub Jelinek  2013-01-31 
13:05:52 UTC ---

Yeah, looks nice.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #12 from Marc Glisse  2013-01-31 
16:53:42 UTC ---

Ok, I'll add a testcase and test the patch in a couple weeks if nobody beats me

to it (I am leaving tonight).


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-01-31 Thread paolo.carlini at oracle dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #13 from Paolo Carlini  2013-01-31 
16:59:52 UTC ---

Great, thanks.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-08 Thread rguenth at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



Richard Biener  changed:



   What|Removed |Added



   Keywords||rejects-valid

   Priority|P3  |P1

   Target Milestone|--- |4.8.0


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-09 Thread jakub at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #14 from Jakub Jelinek  2013-02-09 
18:42:48 UTC ---

Given this is now P1, could the #c9 patch be committed soon, and perhaps just

defer a testcase for it when Marc returns?


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-12 Thread paolo.carlini at oracle dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



Paolo Carlini  changed:



   What|Removed |Added



 CC||paolo.carlini at oracle dot

   ||com



--- Comment #15 from Paolo Carlini  2013-02-12 
16:53:42 UTC ---

Marc, any news on this?


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-12 Thread paolo.carlini at oracle dot com


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #16 from Paolo Carlini  2013-02-12 
16:59:39 UTC ---

Jakub, if Marc remains unreachable I'll take care of carefully checking and

committing the patch in #c9 first thing tomorrow.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-12 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #17 from Marc Glisse  2013-02-12 
20:55:09 UTC ---

(In reply to comment #15)

> Marc, any news on this?



I landed less than 2 hours ago...

I'll see if I can handle it tomorrow.


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-13 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



--- Comment #18 from Marc Glisse  2013-02-13 
21:58:57 UTC ---

Author: glisse

Date: Wed Feb 13 21:58:53 2013

New Revision: 196034



URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=196034

Log:

2013-02-13  Marc Glisse  



PR libstdc++/56111

* include/std/complex (complex): Undefine.

* include/c_compatibility/complex.h (complex): Only undefine if

 has been included.

* testsuite/26_numerics/complex/56111.cc: New testcase.





Added:

trunk/libstdc++-v3/testsuite/26_numerics/complex/56111.cc   (with props)

Modified:

trunk/libstdc++-v3/ChangeLog

trunk/libstdc++-v3/include/c_compatibility/complex.h

trunk/libstdc++-v3/include/std/complex



Propchange: trunk/libstdc++-v3/testsuite/26_numerics/complex/56111.cc

('svn:eol-style' added)



Propchange: trunk/libstdc++-v3/testsuite/26_numerics/complex/56111.cc

('svn:keywords' added)


[Bug libstdc++/56111] [4.8 Regression] {float,double,long double} complex not accepted anymore

2013-02-13 Thread glisse at gcc dot gnu.org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=56111



Marc Glisse  changed:



   What|Removed |Added



 Status|NEW |RESOLVED

 Resolution||FIXED

 AssignedTo|unassigned at gcc dot   |glisse at gcc dot gnu.org

   |gnu.org |



--- Comment #19 from Marc Glisse  2013-02-13 
22:02:33 UTC ---

Done. Users should still convert their code to use _Complex instead if they

want it to work in C++11.