mdebray/tvl-depot
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Prefer start-process to shell-command

Browse source 
Without doing any benchmarking (break this naughty habit), I'm preferring to
call `start-process` instead of `shell-command` in my `wallpaper/set`
function. I noticed that the `shell-command` call was unnecessarily polluting my
`pstree` call when I debugging my randomly changing wallpaper bug.

I'm mostly likely going to change a few more `shell-command` calls to prefer
`start-process`.
This commit is contained in:
William Carroll 2019-12-23 11:33:15 +00:00
parent 4e454f0466
commit 5fbe860b95
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
configs/shared/.emacs.d/wpc/wallpaper.el
View file

@ -31,8 +31,15 @@
"My preferred computer wallpapers.")
(defun wallpaper/set (path)
"Set computer wallpaper to image at `PATH' using `feh` under-the-hood."
(shell-command (string/format "feh --bg-scale --no-fehbg %s" path)))
"Set computer wallpaper to image at `PATH' using `feh` under-the-hood.
`PATH' can be absolute or relative since `f-expand' is called in the function
body to ensure feh can resolve the path."
(start-process "*feh<wallpaper/set>*"
nil
"feh"
"--bg-scale"
"--no-feh-bg"
(f-expand path)))
(defun wallpaper/whitelist-set (wallpaper)
"Focuses the WALLPAPER in the `wallpaper/whitelist' cycle."