Description

thautwarm

opened on Feb 23, 2020

Issue body actions

https://github.com/purescript/purescript-effect/blob/master/src/Effect.js#L12

    return function () {
      return f(a())();
    };

where f is a purescript function. PureScript functions are curried, hence they must accept one and only one argument, but here they call it without arguments.

To speed up calling functions and making FFI easier, I think it could be better for each argument of purescript function has a default value () or None.

Activity

1. lfkdsk commented on Feb 26, 2020

   

   lfkdsk

   on Feb 26, 2020

   Collaborator

   More actions

   @thautwarm we could add default value in PySExpr (when gen define).
2. thautwarm commented on Feb 27, 2020

   

   thautwarm

   on Feb 27, 2020

   MemberAuthor

   More actions

   @lfkdsk We have default values, and we re now using this for pspy.
   See https://github.com/purescript-python/purescripto/blob/2598fa4cbe276db19235b308fedbd817fc89c090/purescripto/workaround.py#L9
3. thautwarm

   added a commit that references this issue on Mar 4, 2020

   0. codegen behavior: add default value `None` for each function argum…

   This issue will close once commit 7d2ab0f is merged into the 'master' branch.

   7d2ab0f

Issue actions