aboutsummaryrefslogtreecommitdiff
path: root/drw.c
Commit message (Collapse)AuthorAge
* introduce drw_fontset_getwidth_clamp()NRK2022-03-25
| | | | | | | | | | | | | getting the width of a string is an O(n) operation, and in many cases users only care about getting the width upto a certain number. instead of calling drw_fontset_getwidth() and *then* clamping the result, this patch introduces drw_fontset_getwidth_clamp() function, similar to strnlen(), which will stop once we reach n. the `invert` parameter was overloaded internally to preserve the API, however library users should be calling drw_fontset_getwidth_clamp() and not depend upon internal behavior of drw_text().
* drw_text: improve both performance and correctnessNRK2022-03-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this patch makes some non-trivial changes, which significantly improves the performance of drawing large strings as well as fixes any issues regarding the printing of the ellipsis when string gets truncated. * performance: before there were two O(n) loops, one which finds how long we can go without changing font, and the second loop would (incorrectly) truncate the string if it's too big. this patch merges the overflow calculation into the first loop and exits out when overflow is detected. when dumping lots of emojies into dmenu, i see some noticeable startup time improvement: before -> after 460ms -> 360ms input latency when scrolling up/down is also noticeably better and can be tested with the following: for _ in $(seq 20); do cat /dev/urandom | base64 | tr -d '\n' | head -c 1000000 echo done | ./dmenu -l 10 * correctness: the previous version would incorrectly assumed single byte chars and would overwrite them with '.' , this caused a whole bunch of obvious problems, including the ellipsis not getting rendered if then font changed. in addition to exiting out when we detect overflow, this patch also keeps track of the last x-position where the ellipsis would fit. if we detect overflow, we simply make a recursing call to drw_text() at the ellipsis_x position and overwrite what was there. so now the ellipsis will always be printed properly, regardless of weather the font changes or if the string is single byte char or not. the idea of rendering the ellipsis on top incase of overflow was from Bakkeby <bakkeby@gmail.com>, thanks! however the original patch had some issues incorrectly truncating the prompt (-p flag) and cutting off emojies. those have been fixed in here.
* Revert "Improve speed of drw_text when provided with large strings"Hiltjo Posthuma2021-08-20
| | | | | | | This reverts commit c585e8e498ec6f9c423ab8ea07cf853ee5b05fbe. It causes issues with truncation of characters when the text does not fit and so on. The patch should be reworked and properly tested.
* Improve speed of drw_text when provided with large stringsMiles Alan2021-08-09
| | | | | | Calculates len & ew in drw_font_getexts loop by incrementing instead of decrementing; as such avoids proportional increase in time spent in loop based on provided strings size.
* Fix memory leaks in drwHiltjo Posthuma2020-06-11
| | | | | Synced from dwm. Patch by Alex Flierl <shad0w73@freenet.de>, thanks.
* Prepared 4.9 release.Anselm R Garbe2019-02-02
|
* drw: drw_scm_create: use Clr typeHiltjo Posthuma2017-11-03
| | | | in this context XftColor is a too low-level type.
* die() consistency: always add newlineHiltjo Posthuma2016-08-12
|
* import new drw from libsl and minor fixes.Markus Teich2016-06-03
| | | | | | | | | - extract drawitem function (code deduplication) - fix bug where inputw was not correctly calculated from the widest item, but just from the one with the longest strlen() which is not the same. It's better now, but does not account for fallback fonts, since it would be too slow to calculate all the correct item widths on startup. - minor code style fixes (indentation, useless line breaks)
* drw: cleanup drw_text, prevent gcc warning false-positive of unused varHiltjo Posthuma2015-10-20
| | | | | ... we don't allow passing text is NULL anymore either, for that behaviour just use drw_rect() (it is used in dwm).
* drw: simplify drw_font_xcreate and prevent a potential unneeded allocationHiltjo Posthuma2015-10-20
|
* drw: a valid (non-NULL) Drw and Fnt context must be passedHiltjo Posthuma2015-10-20
| | | | | don't do these checks on this level. However for resource drw_*_free we will allow it.
* add sbase-style ecalloc(), calloc: or dieHiltjo Posthuma2015-10-20
| | | | ... remove intermediary variables
* drw style improvementsHiltjo Posthuma2015-09-27
| | | | this makes the code-style more consistent aswell.
* Use libdraw: add Xft and fallback-fonts support to graphics libHiltjo Posthuma2015-06-27
- libdraw, util: add drw.{c,h}, util.{c,h} and update code. - libdraw: fix drw_rect(): use w and h parameter. - libdraw: print errstr if last character in string was ":" (sbase). - libdraw: drw_clr_free() allow valid free(NULL). - config.def.h: set default font to monospace. - cleanup() on exit. - LICENSE: update license string for dmenu -v to 2015. - LICENSE: add myself to LICENSE