<div dir="ltr">Thanks Anton,<br><br>I've pushed the patch.<br><br>Unfortunately I've made a bit of a mess of the history by including both patches, and then needing to address the conflicts with another patch.<br><br>Regards,<br>Adam<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 4, 2015 at 3:36 AM, Anton Pirogov <span dir="ltr"><<a href="mailto:anton.pirogov@gmail.com" target="_blank">anton.pirogov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class="">Hi Adam,<br><br>thank you for your suggestions! Indeed I didn't need those instances and I changed the message to your more flexible idea!<br><br>I
 don't know what is the right way to do it here and with darcs in 
general, I used "darcs obliterate --last 1" and recorded a new patch. 
Here is the updated patch, I hope it is ok now.<br><br>Regards,<br>Anton</span><div class="gmail_extra"><br><div class="gmail_quote"><span class="">2015-03-04 1:03 GMT+01:00 adam vogt <span dir="ltr"><<a href="mailto:vogt.adam@gmail.com" target="_blank">vogt.adam@gmail.com</a>></span>:<br></span><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Anton,<br>
<br>
Do you actually need the Read/Show instances? I think it's better to<br>
put a function in the message:<br>
<br>
data ModifySpacing = ModifySpacing (Int -> Int)<br>
<br>
And possibly export these convenience functions:<br>
<br>
setSpacing n = sendMessage (ModifySpacing (\_ -> n))<br>
incSpacing n = sendMessage (ModifySpacing (+n))<br>
<br>
Regards,<br>
Adam<br>
<div><div><br>
On Tue, Mar 3, 2015 at 4:09 PM, Anton Pirogov <<a href="mailto:anton.pirogov@gmail.com" target="_blank">anton.pirogov@gmail.com</a>> wrote:<br>
> This is such a small change that I think that it makes more sense to patch<br>
> it, instead of rolling my own. It just adds the message, nothing more, so it<br>
> does not break anything.<br>
><br>
> Thanks!<br>
><br>
> Best regards,<br>
> Anton Pirogov<br>
><br>
</div></div>> _______________________________________________<br>
> xmonad mailing list<br>
> <a href="mailto:xmonad@haskell.org" target="_blank">xmonad@haskell.org</a><br>
> <a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad</a><br>
><br>
</blockquote></div></div></div><br></div></div>
<br>_______________________________________________<br>
xmonad mailing list<br>
<a href="mailto:xmonad@haskell.org">xmonad@haskell.org</a><br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad</a><br>
<br></blockquote></div><br></div>