<div dir="ltr"><div class="gmail_quote">On Tue, Mar 3, 2015 at 3:39 PM Norbert Zeh <<a href="mailto:nzeh@cs.dal.ca">nzeh@cs.dal.ca</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Dec 15, 2014 at 11:50 AM, Brandon Allbery <span dir="ltr"><<a href="mailto:allbery.b@gmail.com" target="_blank">allbery.b@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"><div class="gmail_extra"><div class="gmail_quote"><span>On Fri, Dec 12, 2014 at 12:10 PM, Norbert Zeh <span dir="ltr"><<a href="mailto:nzeh@cs.dal.ca" target="_blank">nzeh@cs.dal.ca</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">... and thought I'd share.  Patch is attached.<br></div></blockquote><div><br></div></span><div>A few nits:</div><div><br></div><div>- the comment in XMonad.Layout.Dwindle identifies it as XMonad.Layout.Spiral</div><div><br></div><div>- consider renaming Rotation, as anyone using your module must also import XMonad hiding (Rotation). At the very least, you should document the requirement if you're going to keep it, since you export it. Possible alternate name: Chirality.</div></div></div></div></blockquote></div></div></div></blockquote><div><br></div><div>Updated patch is attached.  Let me know if anything else needs to be fixed.</div><div><br></div><div>Cheers,</div><div>Norbert </div></div></div>