Yes, it works better. Thanks for fixing my dumbness. But then, why is it documented the other way ?

Regards,
Arnaud

On Thu, Aug 25, 2011 at 4:30 PM, Roel van Dijk <vandijk.roel@gmail.com> wrote:
Does it help if you format it like this:

-- |Apply a given function over all elements of a list and select one of the
-- results.
selector :: (Arbitrary b)
        => [a]       -- ^ Elements to select from
        -> (a -> b)  -- ^ Constructor to apply to selected element
        -> Gen b
selector list ctor = oneof (map (return . ctor) list)

So placing the => and -> before the lines instead of after them?

2011/8/25 Arnaud Bailly <arnaud.oqube@gmail.com>:
> Hello,
> while trying to generate Haddock documentation (Haddock version 2.9.2, (c)
> Simon Marlow 2006), it chokes on the following fragment
>
> -- |Apply a given function over all elements of a list and select one of the
> -- results.
> selector :: (Arbitrary b) =>
>             [a]      -> -- ^ Elements to select from
>             (a -> b) -> -- ^ Constructor to apply to selected element
>             Gen b
> selector list ctor = oneof (map (return . ctor) list)
>
> with the following error message:
>
> '   parse error on input `-- ^ Elements to select from
>
> I am puzzled as this is exactly how arguments are supposed to be documented,
> according to http://www.haskell.org/haddock/doc/html/ch03s02.html#id565220
>
> Any clues ?
>
> Arnaud
>
> _______________________________________________
> Haskell-Cafe mailing list
> Haskell-Cafe@haskell.org
> http://www.haskell.org/mailman/listinfo/haskell-cafe
>
>