On 2014-02-08 18:37:19 -0500, Calvin Morrison wrote:
> I have had a love affair with dwm's config.h. Fortunately I also
> love i3, and also deal with a plethora of other desktops on my day to
> day work. so I created hotkey (1).
>
> Why wasn't there a simplistic way to have hotkeys, dealt with properly, in
> a suckmore manner? I don't want inane bindings for every WM, I don't
> even want my wm controlling my hotkeys! So I wrote it.
A few comments:
- What's with the weird inconsistent indentation? It makes your code
really hard to read;
- You are using system(), which is highly unporspacele and extremely
fragile;
- There is no author attribution for the code taken from dwm, which is
required;
- You also failed to include the license header, which is required by
the MIT license;
- You are using the unporspacele "return 0" when returning from main() --
use EXIT_SUCCESS instead.
I did not look in detail at the code, there may be less issues.
- application/pgp-signature attachment: stored
Received on Sun Feb 09 2014 - 07:07:33 CET