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
Crashes with large test files #11
Comments
Hi Boyeen, Good catch. I'll investigate this issue and see what I can find. |
Hi Ilya, Any further insight? As I'm sure you have found, iOS7 has some horrible bugs. The worst seems to me the poor memory deallocation algorithm. I am certain the bug I reported to you is not in your code, I see it in a regular UITextView with 1000 lines of code. I think it is related to ARC and the ongoing crashes of Springboard, especially in A7 devices. iOS7.1 helped a little but nowhere near enough. BTW I really love the look of CYRTextView and plan to use it in my own app when I can reliably scroll 1000 lines of code. |
Boyeen, did you fix the issue with large files? |
Nope. It's very frustrating. It seems to be a problem in ARC/iOS7. If you On Fri, Jun 6, 2014 at 12:53 AM, altaveron notifications@github.com wrote:
Topcat Games - www.topcatgames.com |
Boyeen, what do you think about the following issue? #17 |
hey @Boyeen , so, I have the same issue and noticed big performances issues when opening a large file , and scrolling. If you shortcut the drawRect method (it only makes the line gutter fancy), the scrolling problem will go away. I'm planning on working on a PR to improve the perf of the drawRect. For the problem of opening big files (it takes a while to process and display it) I'm still investigating what is the issue. |
I noticed this in my own code but also in the demo. Everything works fine for small snippets but a largish file, say 100+ lines of code, allocates more and more memory then ultimately is terminated after scrolling a while. In fairness, I'm convinced this is due to memory leaks with UITextView in iOS7. I'm just wondering if anyone has found a workaround. It makes CYRTextView unusable for my purpose.
The text was updated successfully, but these errors were encountered: