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

Red: Estimate Size of Classes #27

Open
zezax opened this issue May 17, 2023 · 0 comments
Open

Red: Estimate Size of Classes #27

zezax opened this issue May 17, 2023 · 0 comments
Assignees
Labels
Red Regular Expression DFA

Comments

@zezax
Copy link
Owner

zezax commented May 17, 2023

Here's a breakdown of significant lines of code for each class concept. It combines .h, .cpp, and test code from the include, lib and test directories respectively. The Total column should give a rough idea of how much work it is to review.

Class Header CPP Test Total Type Notes
BitSet 379 0 299 678 template, bit twiddling
Budget 24 0 0 24
Compile 10 33 0 43 wrapper
Consts 16 0 0 16
Debug 74 495 156 725 skip review
DefaultMap 29 0 25 54 template
Dfa 134 220 348 702 iterators, algorithm
Except 55 0 0 55
Executable 45 142 77 264 container
Fnv 31 0 15 46 template
Matcher 583 62 675 1320 templates, optimized
Minimizer 118 256 156 530 algorithm
Nfa 148 269 83 500 iterators, operations
Outcome 21 0 0 21
omnibus 0 0 351 351 just tests
Parser 34 185 75 294 tricky
Powerset 42 171 117 330 algorithm
Proxy 117 0 0 117 iterators, template
Red 169 482 449 1100 API much generated
Scanner 48 262 193 503 tricky
Serializer 67 233 73 373 file format
SparseVec 58 0 110 168 template, algorithm
Types 77 0 0 77
Util 45 129 74 248

The Debug stuff isn't used by the library; so, bugs there wouldn't be very concerning. It's used in some unit tests and is available for ad hoc debugging.

@zezax zezax self-assigned this May 17, 2023
@zezax zezax changed the title Estimate Size of Classes Red: Estimate Size of Classes May 17, 2023
@zezax zezax added the Red Regular Expression DFA label May 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Red Regular Expression DFA
Projects
None yet
Development

No branches or pull requests

1 participant