Bug 39676 - std::result_of doesn't work
Summary: std::result_of doesn't work
Status: RESOLVED INVALID
Alias: None
Product: gcc
Classification: Unclassified
Component: libstdc++ (show other bugs)
Version: 4.4.0
: P3 normal
Target Milestone: ---
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-07 13:14 UTC by Piotr Wyderski
Modified: 2009-04-07 15:45 UTC (History)
3 users (show)

See Also:
Host: WinXP/x86/Cygwin
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Piotr Wyderski 2009-04-07 13:14:53 UTC
------------------8<------------------

#include <functional>


int fn(long, float) {

    return 0;
}


int main(int argc, char *argv[]) {

    typedef std::result_of<decltype(fn)>::type type;
    return 0;
}

------------------8<------------------

$ g++ -std=c++0x testcase.cpp
In file included from /opt/gcc-4.4.0-trunk/lib/gcc/i686-pc-cygwin/4.4.0/include/
c++/functional:75,
                 from testcase.cpp:2:
/opt/gcc-4.4.0-trunk/lib/gcc/i686-pc-cygwin/4.4.0/include/c++/tr1_impl/functiona
l: In instantiation of 'std::_Result_of_impl<false, int ()(long int, float)>':
/opt/gcc-4.4.0-trunk/lib/gcc/i686-pc-cygwin/4.4.0/include/c++/tr1_impl/functiona
l:154:   instantiated from 'std::result_of<int ()(long int, float)>'
testcase.cpp:13:   instantiated from here
/opt/gcc-4.4.0-trunk/lib/gcc/i686-pc-cygwin/4.4.0/include/c++/tr1_impl/functiona
l:200: error: 'int' is not a class, struct, or union type

gcc used is:

$ gcc -v
Using built-in specs.
Target: i686-pc-cygwin
Configured with: ../configure --prefix=/opt/gcc-4.4.0-trunk -v --enable-bootstra
p --enable-version-specific-runtime-libs --enable-static --enable-shared --enabl
e-shared-libgcc --with-gnu-ld --with-gnu-as --enable-sjlj-exceptions --enable-la
nguages=c,c++ --disable-symvers --enable-libjava --disable-nls --with-cpu-32=cor
e2 --with-cpu-64=core2 --enable-threads=posix
Thread model: posix
gcc version 4.4.0 20090309 (experimental) (GCC)
Comment 1 Paolo Carlini 2009-04-07 13:19:30 UTC
Note that *most* of the facilities in <functional> are still following the TR1 specifications, thus, in general, do not expect conformance to the latest C++0x draft (or file a DR for each unimplemented behavior ;)
Comment 2 Piotr Wyderski 2009-04-07 14:05:31 UTC
(In reply to comment #1)
> Note that *most* of the facilities in <functional> are still following the TR1
> specifications, thus, in general, do not expect conformance to the latest C++0x
> draft (or file a DR for each unimplemented behavior ;)
>

The only C++0x functionality involved I am aware of is decltype,
which is outside of the template scope. BTW, 

boost::function_traits<...>::result_type

works flawlessly. Here is a simplified testcase:

#include <functional>

int main(int argc, char *argv[]) {

    typedef std::result_of<int (long, float)>::type type;
    return 0;
}
Comment 3 Paolo Carlini 2009-04-07 14:11:55 UTC
Are you *really* sure the simplified testcase was supposed to work per the TR1 specifications? The author of thar code is Doug Gregor, and it never did in GCC.
Comment 4 Paolo Carlini 2009-04-07 14:15:39 UTC
I'm looking at 3.4/3 in n1836, and my answer is *no*.
Comment 5 Piotr Wyderski 2009-04-07 14:31:45 UTC
So it is a purely C++0x bug, as you indicated in your first reply.
Comment 6 Paolo Carlini 2009-04-07 14:34:30 UTC
Yes, a purely unimplemented C++0x feature.
Comment 7 Jonathan Wakely 2009-04-07 14:45:50 UTC
I think you have the syntax wrong, if you want to know the return type of a function type (or function pointer, or function reference) you need to say:

  result_of<int (*(long, float))(long ,float)>::type

what you probably want is:

  typedef int (*func_ptr)(long, float);
  result_of<func_ptr(long, float)>:type
Comment 8 Paolo Carlini 2009-04-07 14:47:35 UTC
Yes, that would work, we even have testcases for it. But what about C++0x, Jonathan? Seems strange that only that more convoluted syntax is right...
Comment 9 Jonathan Wakely 2009-04-07 14:51:24 UTC
This is not a bug in C++0x either.  The spec is:

Given an rvalue fn of type Fn and values t1, t2, ..., tN of types T1, T2, ..., TN in ArgTypes, respectively,
the type member is the result type of the expression fn(t1, t2, ...,tN).

So the original testcase has Fn=int and tries to find the result of calling int as a function, which is invalid.

The original testcase should be:

  typedef std::result_of<decltype(fn) (long, float)>::type type;

which tells you the result of invoking something of type decltype(fn) with arguments of type long and float.

Or, to see what happens if you pass it other types:

  typedef std::result_of<decltype(fn) (int, double)>::type type;

This tells you the result of calling e.g. f(0, 0.0), which will result in the arguments being implicitly converted, and as it's a simple function not a function object or other callable type with multiple overloads, the result is always 'int'

I believe this is not a bug.
Comment 10 Paolo Carlini 2009-04-07 14:56:58 UTC
Gosh, thanks Jonathan for the explanation, the key word for me is *overloading*, if overloading were not part of C++ these facilities would probably be more "intuitive" to use. Agreed, let's give Piotr two minutes to review your analysis and then let's close it.
Comment 11 Piotr Wyderski 2009-04-07 15:02:42 UTC
Subject: Re:  std::result_of doesn't work

2009/4/7 jwakely dot gcc at gmail dot com <gcc-bugzilla@gcc.gnu.org>:

> what you probably want is:

In fact I want to copy the return type of a template method restore
and use as another method's return type ina C++0x way. This
works:

       class S {

        template <typename T>
          typename
boost::function_traits<decltype(traits<T>::restore)>::result_type

            restore() {

            return traits<T>::restore(*this);
        }
        };

but the std::result_of-based soludins do not. The lambda-like syntax
also does not work, because "this" is not available at the outer scope:

template <typename T> auto restore() -> decltype(traits<T>::restore(*this)) {

    return traits<T>::restore(*this);
}
Comment 12 Jonathan Wakely 2009-04-07 15:40:45 UTC
See what I wrote at http://gcc.gnu.org/ml/libstdc++/2008-09/msg00124.html under point 3)

You want this:

typename std::result_of< decltype(&traits<T>::restore) (S*) >::type

Paolo, let's close this.
Comment 13 Jonathan Wakely 2009-04-07 15:42:03 UTC
(In reply to comment #12)
> typename std::result_of< decltype(&traits<T>::restore) (S*) >::type

Oops, that should be S& not S*

It's not perfect, but for this case std::result_of works if you get the syntax right :)
Comment 14 Paolo Carlini 2009-04-07 15:45:22 UTC
Agreed ;)
Comment 15 Piotr Wyderski 2009-04-07 15:47:17 UTC
Subject: Re:  std::result_of doesn't work

jwakely dot gcc at gmail dot com <gcc-bugzilla@gcc.gnu.org>:

> You want this:
>
> typename std::result_of< decltype(&traits<T>::restore) (S*) >::type

Thank you! :-)

> Paolo, let's close this.

Agreed. :-)

Best regards, Piotr