vava / i3-emacs

i3 emacs integration

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

i3-client can become nil (fix included)

ThibautVerron opened this issue · comments

Hi,

Thanks for your package!

I have run into some minor issues when i3-client is nil. I don't know what causes it, but replacing (boundp 'i3-client) with (bound-and-true-p i3-client) in the test of i3-get-or-make-client (i.e, treating a nil value in the same way as an unassigned value and spawning a proces in this case) seems to work around the issue.

It would also allow to write the whole function as

(or (bound-and-true-p i3-client)
    (setq i3-client
          (<code to spawn the new client>)))

Thanks for feedback! I'll try to find time to make changes but please feel free to send a pull request and I'll happily merge it in.