-
Notifications
You must be signed in to change notification settings - Fork 171
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support method call without parens? #84
Comments
Personally I think it is necessary. If we want to invite more users from Ruby (which I believe is one main feature of this language) to use Goby then they would expect this feature. However there are still several good reasons if we don't want to support it. For example, improving performance for the parser by only handling parentheses. |
@adlerhsieh I don't think we can gain much performance improvement by dropping this feature since the current performance bottleneck is VM. |
Yeah that makes sense. I'm fine with both, then. You can make the call. |
@adlerhsieh I think I won't support it for now and announce this decision is readme with this PR. If people really want this to be supported we can reconsider it. |
Do you want to close this one since this section in README is added? |
Sure |
Currently any method call in goby needs parens to wrap parameters like
foo(1)
, this is not a big issue in normal cases. However, in convention some method calls likeinclude
,require
in Ruby don't need to wrap params with parentheses, we treat them as a configuration more than a method call.So I need some feedback about if
goby
should support this kind of syntax or we should always use parens. 🙏The text was updated successfully, but these errors were encountered: