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.

Sign up for GitHub

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

Jump to bottom

Working branch of #66 #114

Merged
merged 34 commits into from
Sep 27, 2021
Merged

Working branch of #66 #114

merged 34 commits into from
Sep 27, 2021

Conversation

termoshtt
Copy link
Contributor

@termoshtt termoshtt commented Sep 24, 2021 •
edited
Loading

Working branch to merge #66 for CI

g-rauhoeft and others added 24 commits May 21, 2021 14:19
@g-rauhoeft
Added a module for reserved keywords. It contains a list of all EXPRE…
2584686 
…SS keywords in lowercase and a parser that checks whether the following token is a reserved keyword.
@g-rauhoeft
Made the reserved module available from the parser module.
e307b6c
@g-rauhoeft
Changed the letter parser to recognize upper- and lowercase character…
695ec47 
…s, according to the specification.
@g-rauhoeft
Added a many_till function, which uses nom::multi::many_till but work…
262a62e 
…s with the EsprParser trait.
@g-rauhoeft
Changed the order of operations in attribute_decl, because by allowin…
43828c7 
…g upper- and lowercase characters in IDs makes attribute_id recognize things that are actually redeclared_attributes.
@g-rauhoeft
Added uppercase characters to the entity_decl test cases.
7d953e3
@g-rauhoeft
Replaced many1 with many_till in unique_clause with is_reserved as a …
5901b35 
…termination condition. This allows parsing of UNIQUE without accidentally reading keywords.
@g-rauhoeft
Changed order of operations in parameter type between simple_types an…
9c5d43b 
…d named_types, because by allowing uppercase and lowercase, one swallows the other.
@g-rauhoeft
Merge remote-tracking branch 'upstream/master' into case-sensitivity-fix
82b2bd8
@g-rauhoeft
Added eof combinator to check whether the end of the file was reached…
bd6bf65 
…. This is useful when expecting a keyword in unit tests.
@g-rauhoeft
Replaced call to many0 with many_till_reserved for it to terminate co…
ad0ec37 
…rrectly.
@g-rauhoeft
Reverted unit tests that were changed to try alternatives. Replaced m…
8d5b824 
…any_till with many_till_reserved.
@g-rauhoeft
Changed order of operations in qualifiable factor, because it leads t…
e7fd841 
…o errors when allowing upper and lowercase characters.
@g-rauhoeft
Added a many_till_reserved function that repeats an operation until a…
12749ed 
… reserved keyword is found.
@g-rauhoeft
Changed order of operations in supertype_term, because it leads to er…
7e9d7e0 
…rors when allowing upper and lowercase characters.
@g-rauhoeft
Merge remote-tracking branch 'upstream/master' into case-sensitivity-fix
5d042ad
@g-rauhoeft
Converted additional Nom parsers to Espr parsers.
473baf8
@g-rauhoeft
Changed simple_id to check for reserved keywords. Added test cases to…
881fdbe 
… mirror this.
@g-rauhoeft
Changed many_till_reserved back to many0.
0e69d76
@g-rauhoeft
Changed many_till_reserved back to many0.
2e383b7
@g-rauhoeft
Simplified checks for reserved keywords.
71a0ac9
@g-rauhoeft
Restored order or operations for stmt.
edd22ca
@g-rauhoeft
Restored order of operations, because they don't influence the unit t…
077ac08 
…est results.
@termoshtt
Merge branch 'master' into case-sensitivity-fix
b5cab0e
@termoshtt termoshtt self-assigned this Sep 24, 2021
@CLAassistant
Copy link

CLAassistant commented Sep 24, 2021 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

This was referenced Sep 27, 2021
Merged
Merged
@termoshtt termoshtt marked this pull request as ready for review September 27, 2021 13:47
@termoshtt termoshtt merged commit 0ffc7ae into master Sep 27, 2021
@termoshtt termoshtt deleted the case-sensitivity-fix branch September 27, 2021 14:08
@termoshtt termoshtt mentioned this pull request Sep 27, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@termoshtt termoshtt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@termoshtt @CLAassistant @g-rauhoeft