[Github-comments] [geany/geany] Aggressively cache line height for computing margin sizes (b08ae0f)

Colomban Wendling notifications at xxxxx
Fri Feb 5 13:43:26 UTC 2021


@elextr here you go: f600937dd8f592ff00150867537166b2d86aa4dd.

It's more verbose, but possibly easier to grasp, and indeed doe snot suffer from hash clash. And although it's using a lot more memory it should still be reasonable (`sizeof(void*) + strlen(font) + 2 + 3*sizeof(int)`, plus any padding in struct and font string allocation instead of `sizeof(int)`, which with the font "monospace" is gonna be 31 bytes in my x86_64 Linux, versus 4 for the hash alone).

-- 
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/commit/b08ae0fe65c0942dca20281d6841dc83406d8b5e#commitcomment-46789247
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20210205/3055d45c/attachment-0001.htm>


More information about the Github-comments mailing list