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

Consider lazily expanding entities #31

Closed
ianprime0509 opened this issue Nov 23, 2023 · 1 comment
Closed

Consider lazily expanding entities #31

ianprime0509 opened this issue Nov 23, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@ianprime0509
Copy link
Owner

Depends on #30

#30 will offer some obvious efficiency improvements for Reader (reducing the amount of copying necessary). A further improvement which warrants its own issue is the potential to avoid eagerly expanding entities in text content (as described in #29) and instead offering functions on Event to expand raw content on demand. This would avoid such processing for events where the user doesn't care about the expanded content.

This has quite a bit of overlap with the proposed design of #29, and would probably mean that Reader and Parser would use the same Event structure and common expansion functions.

@ianprime0509
Copy link
Owner Author

Implemented in the design of #36.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant