[BUG] Cannot specity `<const>` or `<in>` in function template arguments

[realgdman]

I'm using library (flecs) with API like

ecs.system<Position, const Velocity>()
.each([] some lambda

And haven't found way to specify that const in template call to system()
Which, I believe, cannot be deduced from parameters of lambda

foo<int>();
foo<const int>(); //error
foo<int const>(); //error
foo<in int>(); //error

https://cpp2.godbolt.org/z/sjMWTq3b7

But according to this comment #425 (comment) , it works on pointers
https://cpp2.godbolt.org/z/Tx8e9KfKc
f<* const int>();

[JohelEGP]

I think it may be a bug in the parser.
For f<const int>(),
it's supposed to parse an expression or type-id before a , or >:

//G template-argument-list:
//G     template-argument-list ',' template-argument
//G
//G template-argument:
//G     # note: < > << >> are not allowed in expressions until new ( is opened
//G     expression
//G     type-id

Or maybe it parses const as an identifier and calls it a day, expression completed.
Just to find that it's not followed by , or >, and fails.

[AbhinavK00]

Some Questions:
How do you expect the behavior of <vec> to be different from <in vec>?
Also, how do you expect this to extend upto <inout vec> and related?

One more thing, I think it works with pointers because cpp2 thinks it's a pointer that doesn't change an object instead of a pointer that can't be reseated. Does that make sense?

[JohelEGP]

How do you expect the behavior of <vec> to be different from <in vec>?
Also, how do you expect this to extend upto <inout vec> and related?

See #425 (comment).

One more thing, I think it works with pointers because cpp2 thinks it's a pointer that doesn't change an object instead of a pointer that can't be reseated. Does that make sense?

That's the other side of #456 (comment).
* can't start an expression,
so it's unambiguously the other production of a template-argument,
namely, a type-id.

[realgdman]

How do you expect the behavior of to be different from ?

Ouch.
Thinking about it, when I call function, I expect f(a) work as f(in a) aka f(const a)
But when I call f<A>() what I should expect? It seems that I expect usual f<A>() and need a way to distinguish from f<const A>().
If that's not the case, what would it be? If in aka const is default, All common templates need to be written f<inout A, inout B>()?

[AbhinavK00]

All common templates need to be written f<inout A, inout B>()

I don't think so, as #425(comment) points out, a parameter direction does NOT operate on a type. And when you think about it,

F<out T>
F<move T>

do not make sense. So we need not to change the types in templates to atleast.
There are two ways after this, either dont allow parameter passing directives on non-type template parameters and call it a day OR think about how all of this should work and allow parameter passing directives on non-type template parameters.

[realgdman]

OK I got where my misunderstanding about <in> coming from.
What I learned about project, it was about
foo(in x); and (inout i:=0) { ... } not allowing const.
But apparently you can write

i: const int = 0;
T: type = { j: const int = 0; }

That means some places use new syntax, some old, and thus <const T> should be accepted.

Hence I found workaround for my issue

i: int = 0;
ci: const int = 0;
foo<decltype(i)>();
foo<decltype(ci)>();
foo<decltype(ci)>(); //careful with last use - std::move(decltype(ci)) remove const

print nonconst, const, nonconst.
https://cpp2.godbolt.org/z/8E8MxjnKd

Regarding <in T>, this can be chalked from issue, and thought as discussion of further unifying syntax.

[JohelEGP]

What I learned about project, it was about foo(in x); and (inout i:=0) { ... } not allowing const. But apparently you can write

i: const int = 0;
T: type = { j: const int = 0; }

That means some places use new syntax, some old, and thus <const T> should be accepted.

That's because they're orthogonal features,
and you can use parameter-directions in various contexts.

You can declare a
variable and by-copy parameter
as (const [pointer to]) const T.

You can declare a parameter as being in, inout, out, copy, move, and forward.

You can declare a return type to be by (some of) those.

You can
pass an argument and return an expression
by (some of) those.

[AbhinavK00]

I also learnt the same thing from this issue, I had fully forgotten about const being in cpp2 lol.

[JohelEGP]

const is definitely being parsed as an expression.
See how const * i32 this isn't transformed like in a type-only context: https://cpp2.godbolt.org/z/46KezxeY6.

main: () = {
  x: const * int = g;
  v<const * int>;
}

auto main() -> int{
  int* const x {g};
  v<const * int>;
}

---

One way to fix this is to introduce alternative grammars that are definitely not something else.
After applying #387 (comment)
(note that this clashes with #463's identifier:type expression, so further disambiguation is required):

+//G unqualified-type-id:
+//G     id-expression
+//G     function-type

+//G definite-type-id:
+//G     type-qualifier-seq unqualified-type-id
+//G     definite-function-type

 //G type-id:
-//G     type-qualifier-seq? id-expression
-//G     type-qualifier-seq? function-type
+//G     type-qualifier-seq? unqualified-type-id

 //G template-argument:
 //G     # note: < > << >> are not allowed in expressions until new ( is opened
+//G     definite-type-id
 //G     expression
 //G     type-id

+//G definite-parameter-declaration:
+//G     parameter-direction unnamed-declaration
+//G     this-specifier? parameter-direction? declaration
+//G     this-specifier parameter-direction? identifier

 //G parameter-declaration:
 //G     parameter-direction? unnamed-declaration
 //G     this-specifier? parameter-direction? declaration
 //G     this-specifier? parameter-direction? identifier

+//G definite-parameter-declaration-list
+//G     '(' definite-parameter-declaration-seq ')'

+//G definite-parameter-declaration-seq:
+//G     definite-parameter-declaration
+//G     definite-parameter-declaration-seq ',' definite-parameter-declaration

+//G definite-function-type:
+//G     definite-parameter-declaration-list throws-specifier? return-list?

 //G function-type:
 //G     parameter-declaration-list throws-specifier? return-list? contract-seq?

 //G return-list:
 //G     '->' type-id

And similarly for #387 (comment):

+//G inconclusive-parameter-declaration:
+//G     unnamed-declaration
+//G     parameter-direction identifier
+//G     identifier

+//G inconclusive-parameter-declaration-list:
+//G     inconclusive-parameter-declaration
+//G     inconclusive-parameter-declaration-list ',' inconclusive-parameter-declaration

+//G inconclusive-type-id:
+//G     '(' inconclusive-parameter-declaration-list ')'

 //G is-as-expression:
 //G     prefix-expression
+//G     is-as-expression is-definite-value-constraint
 //G     is-as-expression is-type-constraint
 //G     is-as-expression is-value-constraint
 //G     is-as-expression as-type-cast
 //GTODO     type-id is-type-constraint

+//G is-definite-value-constraint
+//G     'is' inconclusive-type-id

[JohelEGP]

The suggested complication above
is to make f<const i32>() work
while keeping the grammar context-free.

The alternatives I've considered are (assuming no function type support):

• i32 as const to work and cast i32 to const (inconsistent).
• const type-id to be an expression
  that is semantically required to occur in a context
  where type-id is an alternative production.

[JohelEGP]

Here's a simpler alternative, with support for function types.

Proposal parameter-declaration-list achieves feature parity with Cpp1.
It makes the name of a parameter optional in a declaration or type.

The alternative maintains the status-quo: Parameters of a declaration can't be unnamed.
Introduce the converse for types: A function-type-id whose parameters must be unnamed.

First, define function-type-id:

 //G type-id:
 //G     type-qualifier-seq? id-expression
+//G     type-qualifier-seq? function-type-id

 //G parameter-declaration:
 //G     this-specifier? parameter-direction? declaration
 //G     this-specifier? parameter-direction? identifier
+//G parameter-declaration-type:
+//G     parameter-direction? unnamed-declaration

 //G parameter-declaration-list
 //G     '(' parameter-declaration-seq? ')'
+//G parameter-declaration-type-list
+//G     '(' parameter-declaration-type-seq? ')'

 //G parameter-declaration-seq:
 //G     parameter-declaration
 //G     parameter-declaration-seq ',' parameter-declaration
+//G parameter-declaration-type-seq:
+//G     parameter-declaration-type
+//G     parameter-declaration-type-seq ',' parameter-declaration-type

 //G function-type:
 //G     parameter-declaration-list throws-specifier? return-list? contract-seq?
+//G function-type-id:
+//G     parameter-declaration-type-list throws-specifier? return-list?

Then, to fix this issue (#456), update template-argument:

 //G template-argument:
 //G     # note: < > << >> are not allowed in expressions until new ( is opened
+//G     'const' type-id
+//G     function-type-id
 //G     expression
 //G     type-id

With that:

v<const int, // Type.
  (:int), // Function type.
  ((:int=0))>; // Value `0`.

And to workaround #387 (comment):

x is (x); // Same as today.
x is (:int); // Asks if "`x` has type `(:int)`".
x is ((:int=0)); // Asks if "`x` is `0`".