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

bump crengine: memory fixes, some speedup, background image enhancements #1135

Merged
merged 1 commit into from
Jul 5, 2020

Conversation

poire-z
Copy link
Contributor

@poire-z poire-z commented Jul 5, 2020

Includes:


This change is Reviewable

Includes:
- Cleanup some files permissions
- (Upstream) Font: improve management of HarfBuzz features
- (Upstream) Fix invalid cache usage
- (Upstream) Fix many memory errors, allow boxing when cache file
- Disable hasCacheFile()/setBoxingWishedButPreventedByCache()
- globalHash: exclude some settings on initial loading
- CSS: optimize pseudoclass nth-child() and friends
- (Multiple) Fix/silence/avoid clang warnings
- Don't mangle source pixel when alpha-blending to 8bpp (koreader#352)
- (Upstream) HyphMan cleanup
- Rendering methods: remove erm_runin
- Text formatting: simplify 'display: run-in' handling
- Simplify background image drawing
- CSS: adds support for "background-size" property
- CSS: support background:url("data:image/png;base64,...)
- Fix elements cancelling inherited "white-space: pre"
- initNodeStyle(): skip some possibly costly validation
@poire-z
Copy link
Contributor Author

poire-z commented Jul 5, 2020

(How come Travis CI checks now run in 10/15mn ?! I was used to 50/60 minutes...)

@poire-z poire-z merged commit 62bb795 into koreader:master Jul 5, 2020
@poire-z poire-z deleted the bump_crengine branch July 5, 2020 20:48
@Frenzie
Copy link
Member

Frenzie commented Jul 5, 2020

@poire-z As part of #1040 I dropped most of the things it does since those are in CircleCI. I left a few things that probably aren't strictly necessary, like the debug build and the clang + gcc build, which are uniquely on Travis.

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