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

Describe memory management (arena) + more complete parser + std.log #8

Merged
merged 52 commits into from
Mar 31, 2024
Merged

Describe memory management (arena) + more complete parser + std.log #8

merged 52 commits into from
Mar 31, 2024

Conversation

malcolmstill
Copy link
Owner

No description provided.

@malcolmstill
Add all files to main.zig
8ffb155
@malcolmstill
Rename builder convert to toDatalog
8101ae4
@malcolmstill
datalog convert rename to remapSymbols
0aedfda
@malcolmstill
remapSymbols -> remap
e224e17
@malcolmstill
Clean up
8781af0
@malcolmstill
Replace std.mem.starsWith with parser.startsWith
526383e
@malcolmstill
Replace expect / expectString with expect that just takes strings
5c17672
@malcolmstill
More parser clean up
a0235d7
@malcolmstill
More parser clean up
55d4fce
@malcolmstill
Rename fn expect -> fn consume
5b40aab
@malcolmstill
More cleanup
694cc4b
@malcolmstill
More clean up
d2cb81c
@malcolmstill
Split out authorize() functions
bc6ad3b
@malcolmstill
More clean up
33fd7ef
@malcolmstill
Clean up
6900188
@malcolmstill
Clean up
f4d2e44
@malcolmstill
Clean up
eb47527
@malcolmstill
Clean up
2d4e112
@malcolmstill
More clean up
7690bdc
@malcolmstill
Parser fixes
95520a0
@malcolmstill
More parser tests
cdd1222
@malcolmstill
Clean up
59e57be
@malcolmstill
More parser work
120181b
@malcolmstill
Fix number parsing
eae0ad3
@malcolmstill
Number parsing
d222bae
@malcolmstill
Boolean test
8fe6dab
@malcolmstill
Parse bytes
fb9a013
@malcolmstill
Test with all (apart from set) terms
a576a78
@malcolmstill
Set parsing
28c043c
@malcolmstill
More parser
3835b60
@malcolmstill
Spec compliant (?) name parsing
19163c8
@malcolmstill
Clean up
e3d5e06
@malcolmstill
: _ test
9475045
@malcolmstill
Fix variable name parsing + rule parsing tests
e8bd75b
@malcolmstill
More parser tests
5c7dd79
@malcolmstill
Sets can be empty
f28d088
@malcolmstill
fn expr -> fn expr0
c3f9b13
@malcolmstill
Prefer if (!parser.startsWithConsuming(",")) break;
8bbb276
@malcolmstill
Consistent parsing of at least one of N
e01d877
@malcolmstill
Consistent parsing of at least one of N
9863950
@malcolmstill
Policy parsing
9404968
@malcolmstill
parser.temporary()
bdb6ae7
@malcolmstill
Expression parsing tests
cca2bbd
@malcolmstill
Actually, group the binaryOp functions
bd53fd7
@malcolmstill
Fix precedence, fix chained methods
86ad555
@malcolmstill
Clean up
0f2ab5e
@malcolmstill
Clean up
1385108
@malcolmstill
Clean up
ee7973c
@malcolmstill
toDatalog
1d1dbf3
@malcolmstill
Reinstates deinit in datalog structures
faa4405 
- I'm in two minds
- These are being used in tests but not in the actual code, because
  we're depending on arena allocation and so don't need to deinit
- At the same time it would be nice not to do the arena setup in tests
- To some extent it would be nice to know on the individual structures
  level that we know we can deinit
- But it would also be nice _not_ to have the deinit functions to emphasize
  we're using arena
@malcolmstill
FIx scope parsing
c2d833e
@malcolmstill
Rename deinit functions to testDeinit and comment out any deinit call…
83b2d5f 
…s from other modules
@malcolmstill malcolmstill changed the title Add all files to main.zig Clean up parser Mar 31, 2024
@malcolmstill malcolmstill changed the title Clean up parser Describe memory management (arena) + more complete parser + std.log Mar 31, 2024
@malcolmstill malcolmstill merged commit 9aaff35 into master Mar 31, 2024
6 checks passed
@malcolmstill malcolmstill deleted the malcolm/clean-up branch March 31, 2024 03:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@malcolmstill