Inconsistent behaviour between return values of proc calls() and operators (like[])

[dreyer]

Hi alltogether! Using proc definitions, one can do something like this:

> proc huhu()
. {
. print(17);
. }
> proc hihi()
. {
. return(huhu);
. }
> hihi()();
17

In constrast, the following does not work:

> proc huhu()
. {
. print(17);
. }
> list ll=(huhu);
> ll[1]();
// libname  :
// procname : huhu
// type     : singular
   ? error occurred in or before STDIN line 16: `ll[1]();`
   skipping text from `)` error at token `)`

It shouldn't make a difference, whether one gets a callable object (like proc) via another proc or using an operation. BTW, the following works as expected:

> def ll_1=ll[1];
> ll_1();
17

The issue occurs on all objects, which are able to perform the '(' operation.

Best regards,

Alexander

[hannes]

grammar modified to allow such wild constructions

[hannes]

not that easy: had to undo the changes

[hannes]

fixed in the following sense:

proc p {"p";}
list l=p;
proc(l[1])();

Note, that proc(...) is necessary for calling procedures in lists.

[anonymous]

Maybe, the example was to simplistic. The new solution works for proc only, see the overloaded attrib operator for a blackbox object from the pyobject branch:

> pyobject pystr="pypyp";
found bb:pyobject:529 (table:0)
bb-type 529
bb-assign: bb=7f2dfc9b0030
> pyobject(attrib(pystr, "count"))("y");
found bb:pyobject:529 (table:0)
<built-in method count of str object at 0x7f2dfc919930>
   ? error occurred in or before STDIN line 2: `pyobject(attrib(pystr, "count"))("y");`
   ? last reserved name was `attrib`
   skipping text from `"` error at token `)`

PS: The first attempt fixed that issue for me.

[anonymous]

Making that example more like above:

> python_run("def func(): return 17");
> func();
17
> list ll=(func);
> pyobject(ll[1])();
found bb:pyobject:529 (table:0)
<function func at 0x7fae781f3500>
   ? error occurred in or before STDIN line 4: `pyobject(ll[1])();`
   skipping text from `)` error at token `)`

But again, one needs the pyobject branch.

[dreyer]

Ok, got it, the blackbox also needs the proc operation.

So

> proc(ll[1])();
17

is fine