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

Use generators in the parser #24

Open
pR0Ps opened this issue Dec 1, 2014 · 1 comment
Open

Use generators in the parser #24

pR0Ps opened this issue Dec 1, 2014 · 1 comment

Comments

@pR0Ps
Copy link
Member

pR0Ps commented Dec 1, 2014

A lot of the data the parser gives back is gone through with for x in data loops.

Making the parser yield items instead of having to initially compute all of them and store them in memory would probably be more efficient.

@uniphil
Copy link
Member

uniphil commented Dec 1, 2014

Awesome. I would love to see a comparisson of the mem usage before/after this change! Maybe I'll have some time to try it next weekend, if no one gets to it first.

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

No branches or pull requests

3 participants