Ross Paterson <[EMAIL PROTECTED]> writes:
> > libraries/base/Data Set.hs
> > Log:
> > Add an instance of Read to correspond to the existing Show instance.
> > (Implementation based on H'98 Report definition of 'readList'.)
>
> Shouldn't that be reads rather than readsPrec i on the RHS, and the
> whole wrapped in readParen False?
I'm not sure about the precedence thing, so yes perhaps it should be
'reads'. I don't think it needs a readParen though? The curly braces
already delimit the extent of the value, and since the instance of Show
does not introduce parens, I didn't see any need to parse them back in.
If I've missed some subtlety, do just go ahead and fix it.
Regards,
Malcolm
_______________________________________________
Cvs-libraries mailing list
[EMAIL PROTECTED]
http://www.haskell.org/mailman/listinfo/cvs-libraries
