<div dir="ltr"><div>Hi Joachim,</div><div><br></div>I am not sure how to proceed here. I think we need sort of "so be it" recommendation for such cases that are<div>* no big deal</div><div>* already implemented</div><div>* there is no strong opposition (as it looks like)<br><br>Maybe the easiest solution is to close corresponding pull request silently, since the current implementation is not a committee process artifact anyway.</div><div><br></div><div><br></div><div>Vitaly<br><br><br><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Aug 17, 2018 at 9:56 AM Eric Seidel <<a href="mailto:eric@seidel.io">eric@seidel.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Aug 17, 2018, at 12:10, Richard Eisenberg wrote:<br>
> I worry that everyone has missed the final sentence of the proposal:<br>
> <br>
> > This is already implemented, but it is easy enough to tweak the design.<br>
> <br>
> This proposal is fully implemented in 8.4, and I believe it is, too, in <br>
> 8.2. At the time the proposal was written, the new features had not been <br>
> released, and it was hoped that the discussion would influence the <br>
> design. But due to the fact that this was parked for so long, the <br>
> implementation has since been released.<br>
<br>
Oh? I understood "already implemented" to mean on a branch somewhere waiting to be merged.<br>
<br>
> The big motivation for (A) is that it removes unboxed tuples (resp. <br>
> sums) from being a special case. Now, we're uniform.<br>
<br>
Uniformity is a valuable principle. Another valuable principle is the Principle of Least Surprise, which I fear (A) runs afoul of, especially given the public discussion. That's my primary concern wrt. (A). The Manual states that the purpose of unboxed tuples is to return multiple values in registers or on the stack. Boxing the tuples for a lazy pattern match violates this purpose.<br>
<br>
Here's another alternative: keep the uniform behavior that is now implemented, and add a warning for lazy matches on unboxed tuples. I think I'd be happy with this warning. Yes, it adds a bit of syntactic overhead, but it also forces the programmer to clarify what's clearly a subtle issue!<br>
<br>
On the other hand, perhaps the fact that the change is actually in 8.4 and GHC is (presumably) not getting tickets about unexpectedly boxing unboxed-tuples means that this fuss is much ado about nothing, and GHC is clever enough to avoid the boxing.<br>
<br>
> The big motivation for (B) is that it allows more (correct) programs to <br>
> compile. The previous behavior was throwing out programs because GHC was <br>
> worried that the author was being silly -- but that's precisely what <br>
> warnings are good for.<br>
<br>
Indeed, and I fully support (B) regardless of the outcome for (A).<br>
_______________________________________________<br>
ghc-steering-committee mailing list<br>
<a href="mailto:ghc-steering-committee@haskell.org" target="_blank">ghc-steering-committee@haskell.org</a><br>
<a href="https://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-steering-committee" rel="noreferrer" target="_blank">https://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-steering-committee</a><br>
</blockquote></div></div></div>