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

Bug in parsing curly braces #229

Closed
GiovanniBussi opened this issue Apr 28, 2017 · 0 comments
Closed

Bug in parsing curly braces #229

GiovanniBussi opened this issue Apr 28, 2017 · 0 comments
Assignees
Milestone

Comments

@GiovanniBussi
Copy link
Member

I just found a couple of subtle bugs in the parser. PLUMED is not able to parse correctly this

g: GROUP ...
ATOMS={
1
2
}
UNIQUE
...

since there are problems when a closed brace is the single character of the line. Additionally (unrelated) also this is not correctly parsed

g: GROUP ...
ATOMS={
1
2
} UNIQUE
....

Now this is because if there is no space before the curly brace the next word is aggregated with ATOMS.

It is easy to trigger these problems playing with the new @replicas: syntax (that's how I discovered them), so I will make sure they are fixed before the tutorial.

@GiovanniBussi GiovanniBussi added this to the Version 2.3 milestone Apr 28, 2017
@GiovanniBussi GiovanniBussi self-assigned this Apr 28, 2017
@GiovanniBussi GiovanniBussi changed the title But in parsing curly braces Bug in parsing curly braces Apr 28, 2017
GiovanniBussi added a commit that referenced this issue Apr 28, 2017
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

1 participant