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

remove old (pre 0.8) .pxo loading code #904

Merged
merged 1 commit into from
Aug 25, 2023

Conversation

Variable-ind
Copy link
Contributor

No description provided.

@OverloadedOrama
Copy link
Member

Some context for this decision. Versions earlier than 0.8 used a different system for .pxo files that changed in version 0.8, which came out back in September 23 2020, almost 3 years ago. Most people use newer versions than this, so it makes little sense to keep the old compatibility code anymore. Removing this code also allows us to properly detect if a pxo that is being loaded is corrupt or not. Prior to this pull request, loading a corrupt pxo file results in a crash.

Copy link
Member

@OverloadedOrama OverloadedOrama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@OverloadedOrama OverloadedOrama merged commit 2dfab46 into Orama-Interactive:master Aug 25, 2023
@Variable-ind Variable-ind deleted the old-pxo-code branch August 28, 2023 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants