Hi

>> Why not:
>> when (wheni `elem` old_pis) $
>>  (do putAnn $ text "Following line added by " <> showPatchInfo wheni
>>      putLine ' ' l)
>
> Hm, I'm not sure I'd put parentheses around it, but whatever.

I wouldn't either, the fact there are parantheses there is a bug:
http://code.google.com/p/ndmitchell/issues/detail?id=108

> So hlint
> is indeed easily updated? Then I guess I'll send you further bad
> idioms as I find them.

Yes, very easily. For example, to add the above hint I added the single line:

hint = (if x then y else return ()) ==> (when x $ y)

In to the hints file. I didn't even have to recompile anything. The
new manual covers adding hints and ignoring things:
http://www.cs.york.ac.uk/fp/darcs/hlint/hlint.htm

> Great. hlint really is a better name - it instantly tells one what it
> does (everyone knows what 'lint' does), and doesn't conflict with
> current names (eg. DrHylo).

Glad to hear it - changing a projects name is always a bit radical,
but worth it in this case.

I think I should probably stop abusing the darcs@ list now, and will
release HLint shortly. If anyone has any further comments feel free to
email me personally, or haskell-cafe@ after the release.

Thanks

Neil
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to