I'd like to have a default functor for a functor parameter in the constructor of a class. As a minimal example I came up with a class which should server as a filter, which filters elements of type T
iif a filter function returns true. The filter function should be provided in the constructor, defaulting to an "accept all" filter function:
template<class T>
class Filter
{
public:
typedef std::function<bool(const T&)> FilterFunc;
Filter(const FilterFunc & f = [](const T&){ return true; }) :
f(f)
{
}
private:
FilterFunc f;
};
I instantiate the template class like the following:
int main() {
Filter<int> someInstance; // No filter function provided (<-- line 19)
}
However, gcc 4.7 doesn't seem to like this piece of code:
prog.cpp: In constructor ‘Filter<T>::Filter(const FilterFunc&) [with T = int; Filter<T>::FilterFunc = std::function<bool(const int&)>]’:
prog.cpp:19:17: internal compiler error: in tsubst_copy, at cp/pt.c:12141
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-4.7/README.Bugs> for instructions.
Preprocessed source stored into /home/g9i3n9/cc82xcqE.out file, please attach this to your bugreport.
What's wrong? Is my code standard conformant (so GCC is really buggy here or hasn't implemented this) or am I doing something wrong?
As a workaround, I currently use a default-constructed std::function
and only call it (where I want to call it) if it was set:
Filter(const FilterFunc & f = FilterFunc) :
f(f)
{
}
// When using it:
void process() {
if (!f || f(someItem)) { // <-- workaround
}
}
This is a bug: the compiler crashed or a fatal internal error occurred while processing the source code, and the message itself is kindly inviting you to treat the error as such:
"Please submit a full bug report, with preprocessed source if appropriate."
Here is another possible workaround:
template<class T>
class Filter
{
public:
typedef std::function<bool(const T&)> FilterFunc;
Filter() { }
Filter(FilterFunc const& f) : f(f) { }
private:
FilterFunc f = [](const T&){ return true; };
};
As a further alternative, GCC supports delegating constructors, which you may want to consider:
#include <functional>
template<class T>
class Filter
{
public:
typedef std::function<bool(const T&)> FilterFunc;
Filter() : Filter([](const T&){ return true; }) { }
Filter(FilterFunc const& f) : f(f) { }
private:
FilterFunc f;
};
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With