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

Invalid syntax for define variable #9

Open
zzak opened this issue Oct 18, 2013 · 2 comments
Open

Invalid syntax for define variable #9

zzak opened this issue Oct 18, 2013 · 2 comments

Comments

@zzak
Copy link
Contributor

zzak commented Oct 18, 2013

If you check this section it mentions defining a variable *8-30-59* like so:

define variable *8-30-59* = make(<time-of-day>, total-seconds: encode-total-seconds(8, 30, 59));

However, I think variable names must start with a non-number because I get a warning when trying this example.

I should also note I'm using Open Dylan 2013.1

Would you accept a patch to fix this and use a valid variable name?

@waywardmonkeys
Copy link
Member

Yep! I'm still thinking about the other patch.

@zzak
Copy link
Contributor Author

zzak commented Oct 18, 2013

@waywardmonkeys np!

There is no rush, I just wanted to send what patches and bugs I found when working through the book.

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

No branches or pull requests

2 participants