[xmonad] one more layout

Konstantin Sobolev konstantin.sobolev at gmail.com
Tue Apr 14 21:50:20 EDT 2009


On Tue, Apr 14, 2009 at 6:38 AM, Roman Cheplyaka <roma at ro-che.info> wrote:
> * Konstantin Sobolev <konstantin.sobolev at gmail.com> [2009-04-03 17:00:36-0400]
> There are several wishes to your code:
> 1. All non-trivial top-level functions should be documented (such as
>   swap, forwardToFocused etc.)

done

> 2. If you use some function from a different module (like
>   'differentiate' from Combo) you shouldn't copy-paste its code;
>   instead prepare a separate patch which adds that function to export
>   list and explain where you use that function in the patch
>   description. Later that function may be also moved to some module in
>   the Util section.

I got rid of broadcastPrivate, now there's only differentiate left.
It's too specific to be used by anything else I think.

>   Again, those functions (differentiate and broadcastPrivate from
>   Combo) are not documented there. Could you please write documentation
>   for them if you use them?

I've added some short description to differentiate.

> 3. You shouldn't advise to import XMonad.Util.WindowProperties; instead
>   you should re-export needed things (or even the whole module) from
>   your module. See XMonad.Layout.IM for example.

done

> Sorry for a late response and thanks for your work!

thank you


More information about the xmonad mailing list