[PATCH] attempted close_and_remove command

Thorsten Wißmann edu at thorsten-wissmann.de
Thu Aug 28 22:40:41 CEST 2014


Hi,

Just some things I changed in your patch before mergin it:

> Subject: [PATCH] Squashed commit of the following:
> […]

This has no real information because the squashed patch is really small.
Furthermore, the first line of the commit-description should describe
the content of the patch (and not how you created the patch...). So this
is more appropriate:

Subject: Add close_and_remove command

I also added the same line as an entry in the NEWS file.

The code looks good. Notice that the behaviour will be strange if the
window does not close immediately. (Maybe this should be added to the
doc...)

The patch is merged as:

82f56f7 Add close_and_remove command

Cheers,
Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 213 bytes
Desc: not available
URL: <https://listi.jpberlin.de/pipermail/hlwm/attachments/20140828/ed41ea31/attachment.sig>


More information about the hlwm mailing list