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

bug - 修复翻页造成的内存泄漏 #53

Open
wants to merge 71 commits into
base: master
Choose a base branch
from

Conversation

HungryManBB
Copy link

原因是:LSYReadView 的 delegate 属性是 strong ,造成 LSYReadViewController 强引用 LSYReaderView ,造成内存泄漏

GGGHub added 30 commits June 1, 2016 14:46
Former-commit-id: 33e45ac
Former-commit-id: 339e2bf
Former-commit-id: 26751d6
Former-commit-id: b2a5ff5
Former-commit-id: 9dd5f75
Former-commit-id: 7dc83ca
Former-commit-id: 08e03f2
Modify .md

Former-commit-id: f7aa28f
Former-commit-id: a295e4e
Former-commit-id: bd58391
Former-commit-id: 936b678
Former-commit-id: 98e9f87
Former-commit-id: df1ed14
Former-commit-id: 52e9d88
Former-commit-id: 89d834f
Former-commit-id: d29c8d4
Former-commit-id: 0bd276f
Former-commit-id: 7538ee0
Former-commit-id: afa33cd
Former-commit-id: c1754ca
Former-commit-id: 7d94141
Former-commit-id: 201b55f
Former-commit-id: 16b56a6
Former-commit-id: adc8712
Former-commit-id: 60a9398
LiSiYuan and others added 26 commits November 18, 2016 18:48
Former-commit-id: d3c1d9f
Former-commit-id: 1d46895
Former-commit-id: 7d8836d
Former-commit-id: 1536c2b
Former-commit-id: cb0da9e
Former-commit-id: afae8cc
Former-commit-id: 240e11e
Former-commit-id: 0360db3
Former-commit-id: b42bf20
Former-commit-id: db98ecc
Former-commit-id: f7f54ca
Fix broken headings in Markdown files

Former-commit-id: a834daf
修改txt分页错误问题

Former-commit-id: 29c628f
Former-commit-id: 594d089
Former-commit-id: 9018140
Former-commit-id: 2e20530
bug - 图片计算高度错误导致的图片展示比例压缩

Former-commit-id: bb9b666
bug - 修复 epub 多图片渲染出现空白页 bug

Former-commit-id: 6fe07ee
@xindizhiyin2014
Copy link

那个strong改为weak后,会造成过度释放的crash啊@hungryAndCode

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.

4 participants