<div dir="ltr"><div>I've been pretty comfortable with using -Wall from the beginning, but I see the point. I think -Wall is still a good reminder to fix unused bindings, imports etc.</div><div>How about having a flag that disables those warnings while in early development? That would serve as a more explicit WIP marker.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">2020年12月10日(木) 11:23 David Feuer <<a href="mailto:david.feuer@gmail.com" target="_blank">david.feuer@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto">Yeah, I think you're missing something. -Wall is great for checking code that's supposed to be "production ready", but it makes some pretty annoying noise when you're deep in development. Probably the most annoying in that context are unused binding warnings (yeah, I'm not using it yet because I'm still writing it and/or other helpers), unused variable warnings (yeah, that'll be used in some case I still need to write), and unused imports (yeah, I've temporarily commented out the code that uses Foo, and I'm pretty sure I'll be needing Bar before I'm done). In that setting, I'm much more likely to want -Wincomplete-patterns (what do I still need to write) and maybe -Wname-shadowing (so I won't have to go back and change a bunch of names later) than full -Wall.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Dec 9, 2020, 9:06 PM Fumiaki Kinoshita <<a href="mailto:fumiexcel@gmail.com" target="_blank">fumiexcel@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div>I mean literally. An article [0] reminded me of the 
fact that I enable -Wall in 99% of time -- and most packages I use have 
it enabled too.<br></div><div>It's well known that -Wall doesn't enable<b> all </b>warnings, but a subset of warnings that</div><div><br></div><div>* are well accepted by the community<br></div><div>* rarely produce false positives<br></div><div><br></div><div>Well,
 they look like good reasons to enable the warnings by default. Same 
goes for -Wcompat, except that it is not as popular as -Wall. Seeing 
potential problems when compiling code is far less of a pain than 
leaving breakages unnoticed.<br></div><div><br></div><div>Am I missing some obvious reason not to do this?<br></div><div><br></div><div>[0] <a href="https://www.snoyman.com/blog/2020/12/haskell-bad-parts-3" rel="noreferrer" target="_blank">https://www.snoyman.com/blog/2020/12/haskell-bad-parts-3</a></div></div></div>
_______________________________________________<br>
Haskell-Cafe mailing list<br>
To (un)subscribe, modify options or view archives go to:<br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/haskell-cafe" rel="noreferrer noreferrer" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/haskell-cafe</a><br>
Only members subscribed via the mailman list are allowed to post.</blockquote></div>
</blockquote></div></div>