-
Notifications
You must be signed in to change notification settings - Fork 4
Release/2.0.2 #697
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
Merged
Merged
Release/2.0.2 #697
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Several Issues
- Added totalCostLimit and countLimit to all NSCache instances - GeneralCache: 50-150MB limit per instance based on device memory - TPPEncryptedPDFDocument: 30-80MB limit for PDF thumbnails - Added cost tracking for all cache setObject calls - Decentralized memory warning handling (each cache manages itself) - Removed redundant clearAllCaches() call from MemoryPressureMonitor - Changed to async memory clearing to avoid blocking main thread This prevents NSMallocException 'Failed to grow buffer' crashes that occur when removeAllObjects() is called on unbounded caches during memory pressure. Limits are generous and won't impact normal usage or downloads.
- PDF Text Extractor: Replaced string += concatenation with array.joined() - Added 50KB block size limit to prevent unbounded text extraction - Pre-allocate array capacity for better performance - Added size guards to break out of loops before hitting memory limits - Audiobook Logger: Added 2MB file size limit to prevent unbounded growth - Truncate log retrieval to last 1MB for large files - Use FileHandle streaming instead of loading entire files into memory These changes prevent NSMallocException '__CFStringHandleOutOfMemory' crashes that occur when string buffers grow too large during PDF parsing or log operations.
Fix/hotfixes
Produce Builld
Resolves layout issues during navigation transitions
Hide nav bar on epub
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release 2.0.2