summaryrefslogtreecommitdiff
path: root/dmenu.c
AgeCommit message (Collapse)Author
2022-03-25avoid redraw when there's no changeNRK
while i was timing the performance issue, i noticed that there was lots of random redrawing going on. turns out there were coming from here; if someone presses CTRL/ALT etc without pressing anything else, nothing will be inserted, so nothing will change. but the code will `break`, go down and do a needless redraw. this patch changes it to simply return if the keypress iscntrl() also avoid potential UB by casting *buf into an unsigned char.
2022-03-25free all allocated items, use %zu for size_tNRK
`items` itself is not checked for NULL as calling free on NULL is defined to be a no-op.
2022-03-25inputw: improve correctness and startup performanceNRK
a massive amount of time inside readstdin() is spent trying to get the max input width and then put it into inputw, only for it to get clamped down to mw/3 inside setup(). it makes more sense to calculate inputw inside setup() once we have mw available. similar to the last patch, i see noticeable startup performance improvement: before -> after 160ms -> 60ms additionally this will take fallback fonts into account compared to the previous version, so it's not only more performant but also more correct.
2022-03-25significantly improve performance on large stringsNRK
this replaces inefficient pattern of `MIN(TEXTW(..), n)` with drw_fontset_getwidth_clamp() instead, which is far more efficient when we only want up to a certain width. dumping a decently sized (unicode) emoji file into dmenu, I see the startup time drop significantly with this patch. before -> after 360ms -> 160ms this should also noticeably improve input latency (responsiveness) given that calcoffsets() and drawmenu() are pretty hot functions.
2022-03-01remove false-positive warning for int comparison as boolHiltjo Posthuma
Reported by Prathu Baronia <prathu.baronia@praton.me>, patch slightly changed. Thanks!
2022-02-08code-style: rm newline (oops)Hiltjo Posthuma
2022-02-08revert using strcasestr and use a more optimized portable versionHiltjo Posthuma
... compared to the old cistrstr(). Thanks for the feedback!
2022-02-07improve performance of case-insensitive matchingHiltjo Posthuma
2021-08-09add support for more keypad keysHiltjo Posthuma
The keypad Enter key was already supported. On some keyboard layouts like my laptop the page-up and page-down key is more comfortable to use. This adds a few lines but no complexity.
2019-03-03revert IME supportHiltjo Posthuma
dmenu will not handle IME support (st will, atleast for now). revert parts of commit 377bd37e212b1ec4c03a481245603c6560d0be22 this commit also broke input focus.
2019-02-12improve xopenim error messageHiltjo Posthuma
die() already prints a newline.
2019-02-12fix crash when XOpenIM returns NULLHiltjo Posthuma
for example when IME variables are set, but the program is not started (yet).
2019-02-04Close when the embedding window is destroyedQuentin Rameau
2019-01-27Use slow path if stdin is a ttydok
If stdin is a tty and dmenu is ran with the fast option then it's impossible to close stdin because the keyboard is already grabbed.
2018-05-25code-style for pledge: check the return code -1, not < 0Hiltjo Posthuma
this is the proper idiom
2018-05-25code-style for pledge(2)Hiltjo Posthuma
feedback from Klemens, thanks
2018-05-25Pledge on OpenBSDHiltjo Posthuma
2018-04-22Fix cursor drawn position with wide glyphsQuentin Rameau
2018-04-22Handle IME inputQuentin Rameau
Thanks to nzl <uruabi@gmail.com> for the patch!
2018-04-22Fix handling of input stringsHiltjo Posthuma
2018-03-13add key bindings for moving to the word start or endQuentin Rameau
Mod1+b/^Left and Mod1+f/^Right
2018-01-04Fix regression in 84a1bc5Hiltjo Posthuma
Reported by Jochen Sprickerhof, thanks! Applied patch with minor change (only initialize `i` for XINERAMA).
2018-01-04Instantiate j var outside #ifdef XINEMARA directive because it is used in ↵Vincent Carluer
loop outside directive
2017-11-03fix a possible free of a uninitialize variable in paste()Hiltjo Posthuma
2017-11-03init colors using SchemeLastHiltjo Posthuma
this makes it slightly easier to add colors to schemes.
2017-11-03Set class name on menu windowOmar Sandoval
WM_CLASS is a standard ICCCM property which is used to identify windows. Window managers and compositors use it to allow per-application configurable behavior.
2016-12-11Revert "fix input text matching"Andrew Gregory
This reverts commit 09d0a36e0370f7ca9bdb171bf93c5ac3131c5a92. Using strncmp with the length of the user input turns it into a prefix match rather than an exact match as it's supposed to be.
2016-11-25Xinerama: correct variable declarations in preprocessor conditionalThomas Gardner
2016-10-14add embedding support with -w optionQuentin Rameau
2016-10-14dmenu.1: group single optionsQuentin Rameau
2016-08-12die() consistency: always add newlineHiltjo Posthuma
2016-07-26fix crash if negative monitor (< -1) was passedHiltjo Posthuma
for example: dmenu -m '-9001'
2016-07-26Revert "Print highlighted input text only on single match"Hiltjo Posthuma
This reverts commit a4053bc4e403ae57343f43b7e363a0911bba5a3a. My bad, was working on the wrong branch and accidently pushed it... *facepalm*
2016-07-26Print highlighted input text only on single matchQuentin Rameau
When the input text fully matches a single item, do not draw the item and highlight the input text to show that it matches an item in opposition to regular input text not matching anything.
2016-07-25Partially revert 44c7de3: fix items text width offset calculationQuentin Rameau
Without this, we discard the item if it's longer than assigned width instead of truncating it.
2016-06-28fix: Do not crash on e.g. dmenu < /dev/nullS. Gilles
2016-06-03import new drw from libsl and minor fixes.Markus Teich
- 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)
2015-12-19Add config option for word delimitersQuentin Rameau
Let the user configure word boundaries other than ' ', only works with the portable character set.
2015-11-08unboolify dmenuHiltjo Posthuma
2015-11-07fix: multimon: always spawn client on first monitor if specified with -m 0Hiltjo Posthuma
This was always broken. Reproduce: focus client on second monitor, spawn dmenu with -m 0. Result: Old wrong behaviour: dmenu spawns on second monitor (focused client). Now: dmenu spawns on specified monitor (first).
2015-10-18free schemes as arrayHiltjo Posthuma
this makes it slightly easier to add custom schemes aswell
2015-10-04fix input text matchingQuentin Rameau
just compare the size of the input string
2015-10-04fix incorrect ordering of match resultsDavide Del Zompo
look for exact matches comparing the user input against the item text
2015-10-04fix paste from clipboard (ctrl+shift+y)Hiltjo Posthuma
from Joshua Lloret on the ML: "attached is a small patch to enable pasting from clipboard, as well as primary. It seems like there was already code in there to allow this, but since there was never any case to match the upper case 'Y', that inline if would always evaluate to false."
2015-09-28minor style fixHiltjo Posthuma
2015-09-28Remove function prototypes and reorder functions accordinglyFRIGN
2015-09-28Untypedef struct itemFRIGN
Adds clarity. Typedefs for structs are definitely a discussion matter, but there's no reason to hide a simple data-structure behind a meaningless typedef.
2015-09-28Fix the conversion from microseconds to nanosecondsQuentin Rameau
2015-09-28Replace deprecated usleep() with nanosleep()Quentin Rameau
2015-09-27dmenu: style improvementsHiltjo Posthuma
- move main to bottom, usage above main. - dont use variable names with function prototypes. - space before if, for, while, etc: 'if(' -> 'if ('. this makes the code-style more consistent