Skip to content
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

NFN: strings in expressions #23

Closed
basilkohler opened this issue Oct 29, 2014 · 5 comments
Closed

NFN: strings in expressions #23

basilkohler opened this issue Oct 29, 2014 · 5 comments
Assignees

Comments

@basilkohler
Copy link
Contributor

Possible syntax: 'str

@tschudin
Copy link
Contributor

On Wed, 29 Oct 2014, Basil Kohler wrote:

Possible syntax: 'str

Isn't this how it is already implemented? At least I made it that way
for "constant identifiers".

When it comes to strings, I think that spaces as part of the string have
to be supported, wherefore we need to go for either ".." or '..'

@basilkohler
Copy link
Contributor Author

We have not really looked into it yet. I have quickly tested it with a "call 2 /foo/bar 'foobar" and the abstract machine did not handle it correctly in that case. So I would say we keep it open until we have tested it some more.

Ofcourse you are absolutely right with the spaces and we will adopt "..." or '...'.

@tschudin
Copy link
Contributor

so far, identifier constants were used internally and I have to figure
out how exporting values from the result stack for doing the upcall to
the outside world works.

On Wed, 29 Oct 2014, Basil Kohler wrote:

We have not really looked into it yet. I have quickly tested it with a "call 2
/foo/bar 'foobar" and the abstract machine did not handle it correctly in that
case. So I would say we keep it open until we have tested it some more.

Ofcourse you are absolutely right with the spaces and we will adopt "..." or
'...'.


Reply to this email directly or view it on
GitHub.[ADxFAkwKTgYeTHvQJltQd_s4004miQ39ks5nIQiugaJpZM4C0XH1.gif]

@blacksheeep
Copy link
Contributor

Should now work with 'string... ' for call expression.
Maybe we also want to overload add etc. for strings?

@basilkohler
Copy link
Contributor Author

'foo bar' works and is also supported by nfn-scala

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants