Bug 4505 – Type literal of pure function pointer inside function signature
Status
RESOLVED
Resolution
WONTFIX
Severity
normal
Priority
P2
Component
dmd
Product
D
Version
D2
Platform
x86
OS
Windows
Creation time
2010-07-25T07:47:43Z
Last change time
2020-08-06T14:30:40Z
Keywords
rejects-valid
Assigned to
No Owner
Creator
bearophile_hugs
Comments
Comment #0 by bearophile_hugs — 2010-07-25T07:47:43Z
D2 code, foo4() shows that you can't define a pure function pointer in a function signature:
pure int sqr(int x) {
return x * x;
}
pure int foo1(TF)(TF func, int x) { // OK
return func(x);
}
pure int foo2(typeof(&sqr) func, int x) { // OK
return func(x);
}
alias pure int function(int) FN;
pure foo3(FN func, int x) { // OK, from Simen kjaeraas
return func(x);
}
pure int foo4(pure int function(int) func, int x) { // line 14, ERR
return func(x);
}
void main() {
assert(foo1(&sqr, 5) == 25);
assert(foo2(&sqr, 5) == 25);
assert(foo3(&sqr, 5) == 25);
assert(foo4(&sqr, 5) == 25);
}
DMD 2.047 prints:
test.d(14): basic type expected, not pure
test.d(14): found 'pure' when expecting ')'
test.d(14): semicolon expected following function declaration
test.d(14): no identifier for declarator int function(int)
test.d(14): semicolon expected, not 'int'
test.d(14): semicolon expected, not ')'
test.d(14): Declaration expected, not ')'
test.d(16): unrecognized declaration
Also, it generates too many error messages.
Comment #1 by andrej.mitrovich — 2010-08-29T19:11:49Z
I think this is a problem of the keyword pure trying to act as a type specifier for the return type ( See also my bug 4734 for a similar issue).
If you put pure after the function definition but before the identifier, like so:
pure int foo4(int function(int) pure func, int x) {
then your example compiles and all asserts pass.
Comment #2 by k.hara.pg — 2015-03-03T03:57:34Z
*** Issue 10603 has been marked as a duplicate of this issue. ***
Comment #3 by pro.mathias.lang — 2020-08-06T14:30:40Z
As mentioned by Andrej, it's possible if you put the keyword on the RHS, which is the recommended way to do it nowadays. I don't think that's an issue worth fixing, so marking as WONTFIX.