[Github-comments] [geany/geany] Slow startup (#2649)

[Colomban Wendling]

OK, took me long enough to work on that one, but please give the branch [`startup-speed`](/geany/geany/tree/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
825acb21eabd33390616b224b0454886dc7d6ec2 | d9f8cdbad58d09f0c18ca8acccb49209263018f0 patched to use fixed 16px-wide margins | d9f8cdbad58d09f0c18ca8acccb49209263018f0 | b08ae0fe65c0942dca20281d6841dc83406d8b5e

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 or view it on GitHub:
https://github.com/geany/geany/issues/2649#issuecomment-773664375
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210204/8bb5288c/attachment.htm>