<div dir="auto">Thanks Richard<div dir="auto"><br></div><div dir="auto">This MR is a huge change,  and hard to digest.  But it is also a step function,  in that we cannot have the old way of using API Annotations for exact printing and the new way at the same time.</div><div dir="auto"><br></div><div dir="auto">So I have focused on making sure that it can actually do what I believe is required, which I am satisfied it now does.   Admittedly this needs to be more clearly defined. I am happy to do that,  and to tweak the implementation based on feedback.  There are plenty of rough spots, as I have been working on the big picture at the expense of polished details.</div><div dir="auto"><br></div><div dir="auto">And I do appreciate the willingness to merge now and then clean up, this will make my life a lot simpler,  I have rebased 50 odd times already.</div><div dir="auto"><br></div><div dir="auto">Regards</div><div dir="auto">  Alan</div><br><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Fri, 12 Mar 2021, 22:21 Richard Eisenberg, <<a href="mailto:rae@richarde.dev">rae@richarde.dev</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word;line-break:after-white-space">After a consult with Simon, I've updated the relevant wiki page at <a href="https://gitlab.haskell.org/ghc/ghc/-/wikis/api-annotations" target="_blank" rel="noreferrer">https://gitlab.haskell.org/ghc/ghc/-/wikis/api-annotations</a> with a sketch of a design description for this new feature, along with lots of questions. Both Simon and I agree that it may be more sensible to merge first and ask questions later, but we do think the design could be tightened in a few places.<div><br></div><div>There are no notifications etc on wiki page updates, so it might be good to also correspond via email when updates take place.</div><div><br></div><div>Richard<br><div><br><blockquote type="cite"><div>On Mar 11, 2021, at 6:48 PM, Richard Eisenberg <<a href="mailto:rae@richarde.dev" target="_blank" rel="noreferrer">rae@richarde.dev</a>> wrote:</div><br><div><div style="word-wrap:break-word;line-break:after-white-space">I've started a review, but sent along what I had when dinner was ready. Hopefully more later, but don't wait up for me!<div><br></div><div>Incidentally: this is a monstrous patch, and so there is a strong incentive just to get on with it without resolving all these quibbles. I won't stand in your way on that front -- it might be better to improve this after it lands. However, I also see quite a few TODO:AZ notes. Are you intending to fix these before landing? Or do you think it's OK to merge first and then return?</div><div><br></div><div>High level piece: I'm in support of this direction of movement -- I just want to make sure that the new code is understandable and maintainable.</div><div><br></div><div>Thanks,</div><div>Richard<br><div><div><br><blockquote type="cite"><div>On Mar 6, 2021, at 12:39 PM, Alan & Kim Zimmerman <<a href="mailto:alan.zimm@gmail.com" target="_blank" rel="noreferrer">alan.zimm@gmail.com</a>> wrote:</div><br><div><div dir="ltr"><div>I have been running a branch in !2418[1] for just over a year to migrate the ghc-exactprint functionality directly into the GHC AST[2], and I am now satisfied that it is able to provide all the same functionality as the original.</div><div><br></div><div>This is one of the features intended for the impending 9.2.1 release, and it needs to be reviewed to be able to land.  But the change is huge, as it mechanically affects most files that interact with the GHC AST.</div><div><br></div><div>So I have split out a precursor !5158 [3] with just the new types that are used to represent the annotations, so it can be a focal point for discussion.</div><div><br></div><div>It is ready for review, please comment if you have time and interest.</div><div><br></div><div>Regards</div><div>  Alan</div><div><br></div><div>[1] <a href="https://gitlab.haskell.org/ghc/ghc/-/merge_requests/2418" target="_blank" rel="noreferrer">https://gitlab.haskell.org/ghc/ghc/-/merge_requests/2418</a></div><div>[2] <a href="https://gitlab.haskell.org/ghc/ghc/-/issues/17638" target="_blank" rel="noreferrer">https://gitlab.haskell.org/ghc/ghc/-/issues/17638</a></div><div>[3] <a href="https://gitlab.haskell.org/ghc/ghc/-/merge_requests/5158" target="_blank" rel="noreferrer">https://gitlab.haskell.org/ghc/ghc/-/merge_requests/5158</a></div></div>
_______________________________________________<br>ghc-devs mailing list<br><a href="mailto:ghc-devs@haskell.org" target="_blank" rel="noreferrer">ghc-devs@haskell.org</a><br><a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" target="_blank" rel="noreferrer">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br></div></blockquote></div><br></div></div></div>_______________________________________________<br>ghc-devs mailing list<br><a href="mailto:ghc-devs@haskell.org" target="_blank" rel="noreferrer">ghc-devs@haskell.org</a><br><a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" target="_blank" rel="noreferrer">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br></div></blockquote></div><br><br></div></div></blockquote></div></div>