You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
OK, took me long enough to work on that one, but please give the branch startup-speed a spin.
My results, which are the mean average time (measuring document_open_file_full()) for opening Geany's src/*.[ch] (96 C files), best out of 3 runs:
| 1.36 (A) | 1.37 fixed-width (B) | ±1.37.1 (C) | startup-speed branch (D) |
|---|---|---|---|
| 0.048028s | 0.068192s | 0.146446s | 0.048415s |
| 825acb2 | d9f8cdb patched to use fixed 16px-wide margins | d9f8cdb | b08ae0f |
mean difference (round(Y * 100 / X)):
| A | B | C | D | |
|---|---|---|---|---|
| A | 100% | 70% | 33% | 99% |
| B | 142% | 100% | 47% | 141% |
| C | 305% | 215% | 100% | 302% |
| D | 101% | 71% | 33% | 100% |
As you can see, my branch is ever so slightly slower than 1.36 -- if the precision here means anything. I'm not sure why 1.37 with fixed-width margins is so much slower than 1.36, maybe a change in Scintilla? Not sure.
Anyway, the conclusion can be that 1.37 is real slow, about 3 times slower than 1.36 in my example. My proposed fix is back to 1.36-level speeds, and keeps the dynamic margin sizes, so I think it's a win-win, although there's indeed a little more code complexity.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.