Hi, I fixed the warnings. I just used xmonad --recompile while developing, will use ghc -Wall before submitting in the future, sorry for the inconvenience! Best regards, Anton 2015-03-15 0:43 GMT+01:00 adam vogt :

Hi Anton,

Would you be able to address the warnings ghc -Wall spits out http://lpaste.net/8937793665569390592 ?

For example you have an incomplete pattern for:

handleResize :: BinarySpacePartition Window -> WindowArrangerMsg -> X (Maybe (BinarySpacePartition Window))

It would be better to write

handleResize :: BinarySpacePartition Window -> Rectangle -> X (Maybe (BinarySpacePartition Window)) handleResize b (newRect @ Rectangle x y w h) = ...

Thanks, Adam

On Thu, Mar 12, 2015 at 11:11 AM, Anton Pirogov wrote:

...

Hi everyone,

here is a patch adding following changes to the BSP layout:

Actions like Swap also affect the windows associated with the leaves, leading to much more intuitive behaviour

Mouse support: through SetGeometry (e.g. MouseResize / BorderResize)

FocusParent action (like in i3): Without this sometimes it is simply impossible to swap the elements one wants (e.g. when having 2 windows on the left and 2 on the right, each in a split and wanting to swap these two window pairs) and in other cases this is more convenient than having to first focus something else to get the desired effect. With it the user can apply actions not only to leafs but also to nodes, thereby being able to manipulate splits that only contain other splits and no leafs (windows)

Equalize and Balance action: Equalize adjusts the ratios so that each window occupies the same space, Balance retiles the layout if it gets too complicated

Since my last mail (which received no reactions) I removed a lot of stuff that was indeed possible with other layout modifiers. This patch contains strictly new features and improvements to the layout.

Best regards, Anton

_______________________________________________ xmonad mailing list xmonad@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad

You can enable warnings by adding a pragma {-# OPTIONS_GHC -Wall #-} On Mar 15, 2015 8:42 AM, "Carsten Mattner" wrote:

On Sun, Mar 15, 2015 at 11:08 AM, Anton Pirogov wrote:

...

Hi,

I fixed the warnings. I just used xmonad --recompile while developing, will use ghc -Wall before submitting in the future, sorry for the inconvenience!

Isn't it possible to pass in an environment variable to extend the GHC flags and enale -Wall during xmonad --recompile? If there is it might be good idea to document it.

...

Best regards, Anton

2015-03-15 0:43 GMT+01:00 adam vogt :

...

Hi Anton,

Would you be able to address the warnings ghc -Wall spits out http://lpaste.net/8937793665569390592 ?

For example you have an incomplete pattern for:

handleResize :: BinarySpacePartition Window -> WindowArrangerMsg -> X (Maybe (BinarySpacePartition Window))

It would be better to write

handleResize :: BinarySpacePartition Window -> Rectangle -> X (Maybe (BinarySpacePartition Window)) handleResize b (newRect @ Rectangle x y w h) = ...

Thanks, Adam

On Thu, Mar 12, 2015 at 11:11 AM, Anton Pirogov <

anton.pirogov@gmail.com>

...

wrote:

...

Hi everyone,

here is a patch adding following changes to the BSP layout:

Actions like Swap also affect the windows associated with the leaves, leading to much more intuitive behaviour

Mouse support: through SetGeometry (e.g. MouseResize / BorderResize)

FocusParent action (like in i3): Without this sometimes it is simply impossible to swap the elements one wants (e.g. when having 2 windows

on the

...

left and 2 on the right, each in a split and wanting to swap these two window pairs) and in other cases this is more convenient than having to first focus something else to get the desired effect. With it the user can apply actions not only to leafs but also to nodes, thereby being able to manipulate splits that only contain other splits and no leafs (windows)

Equalize and Balance action: Equalize adjusts the ratios so that each window occupies the same space, Balance retiles the layout if it gets too complicated

Since my last mail (which received no reactions) I removed a lot of stuff that was indeed possible with other layout modifiers. This patch contains strictly new features and improvements to the layout.

Best regards, Anton

_______________________________________________ xmonad mailing list xmonad@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad

_______________________________________________ xmonad mailing list xmonad@haskell.org http://mail.haskell.org/cgi-bin/mailman/listinfo/xmonad