<br><br><div class="gmail_quote">On Fri, Jul 10, 2009 at 2:08 PM, Jared Jennings <span dir="ltr">&lt;<a href="mailto:jjenning@gmail.com">jjenning@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I&#39;ve written a parser for OFX 1.x (these are the files you can<br>
download from most banks detailing recent account activity).<br>
<br>
I&#39;m not sure I&#39;ve written it well. This is my first serious thing in Haskell.</blockquote><div><br>At a quick glance I can give some stylistic feedback:<br><br>* I like to have at least 1 blank line between functions<br>
* Add type signatures for top level definitions, this helps later on<br>* More haddocks next to function definitions :)<br><br>I didn&#39;t read your code deeply enough to comment on the way you used things, named stuff, built abstractions, etc.<br>
<br>Jason<br></div></div><br>