Remove surrounding $() to avoid executing output

salman-abedin / devour

X11 window swallower

GNU General Public License v2.0

419 stars 30 forks source link

Closed guardam closed 4 years ago

guardam commented 4 years ago

$("$@") > /dev/null 2>&1

Should be replaced with:

("$@" >/dev/null 2>&1)

See SC2091 for more informations.

salman-abedin commented 4 years ago

I knew I was doing something fishy there! Couldn't put my fingers on it. Thanks for pointing it out. Adding you as a contributor.