<div dir="ltr">glad to see this finally moving forward! <br><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 20, 2017 at 6:19 PM, Ted Kremenek via swift-evolution <span dir="ltr">&lt;<a href="mailto:swift-evolution@swift.org" target="_blank">swift-evolution@swift.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">



<div>
<div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif">
<p style="margin-right:0px;margin-bottom:15px;margin-left:0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255);margin-top:0px!important">The review of &quot;SE-0193 - Cross-module inlining and specialization&quot; begins now and runs through <b>January 5, 2018</b>.</p>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">The proposal is available here:</p>
<blockquote style="margin:5px 5px;padding-left:10px;border-left:thin solid #1abc9c">
<p style="margin:0px"><a href="https://github.com/apple/swift-evolution/blob/master/proposals/0193-cross-module-inlining-and-specialization.md" target="_blank">https://github.com/apple/<wbr>swift-evolution/blob/master/<wbr>proposals/0193-cross-module-<wbr>inlining-and-specialization.md</a></p>
</blockquote>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">Reviews are an important part of the Swift evolution process. All review feedback should be sent to the swift-evolution mailing list at:</p>
<blockquote style="margin:5px 5px;padding-left:10px;border-left:thin solid #1abc9c">
<p style="margin:0px"><a href="https://lists.swift.org/mailman/listinfo/swift-evolution" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a></p>
</blockquote>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">or, if you would like to keep your feedback private, directly to the review manager. </p>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">When replying, please try to keep the proposal link at the top of the message:</p>
<blockquote style="margin:5px 5px;padding-left:10px;border-left:thin solid #1abc9c">
<p style="margin:0px">Proposal link: <a href="https://github.com/apple/swift-evolution/blob/master/proposals/0193-cross-module-inlining-and-specialization.md" target="_blank">https://github.com/apple/<wbr>swift-evolution/blob/master/<wbr>proposals/0193-cross-module-<wbr>inlining-and-specialization.md</a><br>
...<br>
Reply text<br>
...<br>
Other replies</p>
</blockquote>
<h3 id="m_788140584857268535toc_0" style="margin:20px 0px 10px;padding:0px;font-size:18px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">What goes into a review of a proposal?</h3>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">The goal of the review process is to improve the proposal under review through constructive criticism and, eventually, determine the direction of Swift. </p>
<p style="margin:15px 0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">When reviewing a proposal, here are some questions to consider:</p>
<ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)">
<li style="margin:0px">
<p style="margin:0px 0px 15px">What is your evaluation of the proposal?</p></li></ul></div></div></blockquote><div>this is a feature i have been waiting for for a long time so needless to say i strongly support this proposal. one comment is that <span style="font-family:monospace,monospace">@abiPublic</span> is a kind of awkward thing to have around, not because of how it’s spelled but because i think access control and abi visibility are orthogonal concepts and i’m not a fan of overloading access control terms for abi concepts. it makes sense to have <span style="font-family:monospace,monospace">@abiPublic</span> on <span style="font-family:monospace,monospace">private</span> and <span style="font-family:monospace,monospace">fileprivate</span> declarations too and i hope this gets added, because <span style="font-family:monospace,monospace">private</span> and <span style="font-family:monospace,monospace">fileprivate</span> are tools for code organization and maintenance,, the compiler with wmo doesn’t care about <span style="font-family:monospace,monospace">private</span> vs <span style="font-family:monospace,monospace">internal</span>. but <span style="font-family:monospace,monospace">@abiPublic private</span> is bound to cause confusion and it just reads funny. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)"><li style="margin:0px">
</li>
<li style="margin:0px">
<p style="margin:0px 0px 15px">Is the problem being addressed significant enough to warrant a change to Swift?</p></li></ul></div></div></blockquote><div>Yes. this issue (along with generic specialization which is really rendered mostly irrelevant by inlining) is the main technical barrier preventing a swift core library ecosystem from taking root. formalizing this feature will allow library authors like me to ship and use modules for low level tasks, whereas previously the workaround was to copy and paste <a href="https://github.com/kelvin13/swiftlets">handy .swift files</a> containing implementations of common data structures and algorithms. ultimately this will help maintainability, code reuse, and general code base cleanliness.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)"><li style="margin:0px">
</li>
<li style="margin:0px">
<p style="margin:0px 0px 15px">Does this proposal fit well with the feel and direction of Swift?</p></li></ul></div></div></blockquote><div>i don’t see why it wouldn’t. the proposal seems overly conservative and leans a bit too far towards maximum resilience vs maximum optimization but that’s fine. <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)"><li style="margin:0px">
</li>
<li style="margin:0px">
<p style="margin:0px 0px 15px">If you have used other languages or libraries with a similar feature, how do you feel that this proposal compares to those?</p></li></ul></div></div></blockquote><div>this is a big step up from the c++ thing where you would just distribute giant “header-only” libraries so<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)"><li style="margin:0px">
</li>
<li style="margin:0px">
<p style="margin:0px 0px 15px">How much effort did you put into your review? A glance, a quick reading, or an in-depth study?</p></li></ul></div></div></blockquote><div>i read the whole thing,, and i’ve been following this discussion for a while <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div name="messageBodySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><ul style="margin:15px 0px;padding-left:30px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255)"><li style="margin:0px">
</li>
</ul>
<p style="margin-top:15px;margin-right:0px;margin-left:0px;font-family:Helvetica,arial,sans-serif;background-color:rgb(255,255,255);margin-bottom:0px!important">Thanks,<br>
Ted Kremenek<br>
Review Manager</p>
</div>
<div name="messageReplySection" style="font-size:14px;font-family:-apple-system,BlinkMacSystemFont,sans-serif"><br>
<div></div>
</div>
</div>

<br>______________________________<wbr>_________________<br>
swift-evolution mailing list<br>
<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a><br>
<a href="https://lists.swift.org/mailman/listinfo/swift-evolution" rel="noreferrer" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-<wbr>evolution</a><br>
<br></blockquote></div><br></div></div></div>