[darcs-devel] [issue1988] Enable -fwarn-unused-do-bind

[Gabriel Kerneis]

New submission from Gabriel Kerneis <kerneis at pps.jussieu.fr>:

Hi,

The following warning is disabled in darcs.cabal: -fwarn-unused-do-bind.

It should be activated, because unused bind in do blocks might hide
errors.  For instance, while resolving issue332, I wanted to change a
function throwing an exception into a function returning IO(Either). 
With the warning disabled, ghc does not tell me where this function is
used, which means I have to use grep, cross fingers and hope I do not
miss any error case.

Currently, this warning triggers 343 times when building darcs.  Who
knows how many bugs are hidden in this crowd?

Of course, enabling the warning is easy but it is essential to fix the
warned cases, in order to get something usable.  This requires a
collective effort, because I think no one could bear fixing more than a
few dozen of warnings at once.

Best regards.

----------
messages: 12902
milestone: 2.8.0 HEAD
nosy: dmitry.kurochkin, kerneis
priority: bug
status: unknown
title: Enable -fwarn-unused-do-bind

__________________________________
Darcs bug tracker <bugs at darcs.net>
<http://bugs.darcs.net/issue1988>
__________________________________