Recent performance regression? · gsantner/markor · Discussion #1794 (original) (raw)

Within the last google play store update, I noticed a performance regression:

I have a 75k markdown file that is very slow when entering text now.

Any thoughts or ideas?

You must be logged in to vote

I have the same problem. Use Markor to edit fairly large plain text files (sometimes 200kb+) and while previous versions had no issue, newest release is unusable. Typing leads to one character inserted per second or so.

You must be logged in to vote

0 replies

200k files with highlighting have always been slow. The more highlighting the slower they will be. The issue is fundamental to how android's TextView works.

Please turn off highlighting and confirm that the performance is better.

My most recent changes add dynamic highlighting support. With them, I am able to edit 500k files just fine.

You must be logged in to vote

0 replies

Actually, on investigating this further, there is definitely something strange going on.

Test files which were really fast to edit a few days ago are now slow. Disabling highlighting and auto format does not fix the issue.

Further, scrolling is nice and fast until I make an edit, and then the scrolling becomes slow. Not sure what could be causing these issues. A quick look at profiling data shows nothing.

You must be logged in to vote

0 replies

@gsantner The performance regression is not present in 1193761, right before master / AndroidX was merged back into partial highlighitng, and is present in bd1f3c8, the merge commit.

Looking at the diff to see what the cause may be. Hopefully its not just that AndroidX AppCompatEditText is slow...

You must be logged in to vote

5 replies

@gsantner I will definitely need help investigating this

Last time I checked the androidx edittext component souece code it looked like very little difference compared to raw EditText.

Read your linked bug, there is some emoji feature to turn off at androidX edittext?! When its really just that and improves/fixes performance..why not 😄

Though, I dont have a Android 12 device, here on A.10 its not present to me. Also the AndroidX stuff was merged to your branch for a week or so as well right (and you didnt notice it either?)

Didn't notice it earlier. I thought all performance issues had been solved.

No worries, and of course not blaming. Thanks for the PR!

Currently have day job to do, I can't program throughout the day mostly.

You must be logged in to vote

0 replies