<p dir="ltr">You can enable warnings by adding a pragma</p>
<p dir="ltr">{-# OPTIONS_GHC -Wall #-}</p>
<div class="gmail_quote">On Mar 15, 2015 8:42 AM, "Carsten Mattner" <<a href="mailto:carstenmattner@gmail.com">carstenmattner@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, Mar 15, 2015 at 11:08 AM, Anton Pirogov <<a href="mailto:anton.pirogov@gmail.com">anton.pirogov@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I fixed the warnings. I just used xmonad --recompile while developing, will<br>
> use ghc -Wall before submitting in the future, sorry for the inconvenience!<br>
<br>
Isn't it possible to pass in an environment variable to extend the<br>
GHC flags and enale -Wall during xmonad --recompile?<br>
If there is it might be good idea to document it.<br>
<br>
> Best regards,<br>
> Anton<br>
><br>
><br>
> 2015-03-15 0:43 GMT+01:00 adam vogt <<a href="mailto:vogt.adam@gmail.com">vogt.adam@gmail.com</a>>:<br>
>><br>
>> Hi Anton,<br>
>><br>
>> Would you be able to address the warnings ghc -Wall spits out<br>
>> <a href="http://lpaste.net/8937793665569390592" target="_blank">http://lpaste.net/8937793665569390592</a> ?<br>
>><br>
>> For example you have an incomplete pattern for:<br>
>><br>
>> handleResize :: BinarySpacePartition Window -> WindowArrangerMsg -> X<br>
>> (Maybe (BinarySpacePartition Window))<br>
>><br>
>> It would be better to write<br>
>><br>
>> handleResize :: BinarySpacePartition Window -> Rectangle -> X (Maybe<br>
>> (BinarySpacePartition Window))<br>
>> handleResize b (newRect @ Rectangle x y w h) = ...<br>
>><br>
>> Thanks,<br>
>> Adam<br>
>><br>
>> On Thu, Mar 12, 2015 at 11:11 AM, Anton Pirogov <<a href="mailto:anton.pirogov@gmail.com">anton.pirogov@gmail.com</a>><br>
>> wrote:<br>
>>><br>
>>> Hi everyone,<br>
>>><br>
>>> here is a patch adding following changes to the BSP layout:<br>
>>><br>
>>> Actions like Swap also affect the windows associated with the leaves,<br>
>>> leading to much more intuitive behaviour<br>
>>><br>
>>> Mouse support: through SetGeometry (e.g. MouseResize / BorderResize)<br>
>>><br>
>>> FocusParent action (like in i3): Without this sometimes it is simply<br>
>>> impossible to swap the elements one wants (e.g. when having 2 windows on the<br>
>>> left and 2 on the right, each in a split and wanting to swap these two<br>
>>> window pairs) and in other cases this is more convenient than having to<br>
>>> first focus something else to get the desired effect. With it the user can<br>
>>> apply actions not only to leafs but also to nodes, thereby being able to<br>
>>> manipulate splits that only contain other splits and no leafs (windows)<br>
>>><br>
>>> Equalize and Balance action: Equalize adjusts the ratios so that each<br>
>>> window occupies the same space, Balance retiles the layout if it gets too<br>
>>> complicated<br>
>>><br>
>>> Since my last mail (which received no reactions) I removed a lot of stuff<br>
>>> that was indeed possible with other layout modifiers. This patch contains<br>
>>> strictly new features and improvements to the layout.<br>
>>><br>
>>> Best regards,<br>
>>> Anton<br>
>>><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>
>><br>
><br>
><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>