<html><head><meta http-equiv="content-type" content="text/html; charset=utf-8"></head><body dir="auto"><div><span></span></div><div><div>There was an exchange in the past on how extensions are very different from swift nominal/structural types, and Doug even shared how adding scoping to extensions is not a trivial task by a long shot. In this context, talking about unifying modifier behaviors does not make much sense to me. I am not sure i understand what motivates this change.</div><div id="AppleMailSignature"><div>Regards</div>(From<span class="Apple-style-span" style="-webkit-tap-highlight-color: rgba(26, 26, 26, 0.294118);"> mobile)</span></div><div><br>On Jun 26, 2016, at 8:59 PM, Adrian Zubarev via swift-evolution <<a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a>> wrote:<br><br></div><blockquote type="cite"><div><style>
body {
        font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;
        padding:1em;
        margin:auto;
        background:#fefefe;
}
h1, h2, h3, h4, h5, h6 {
        font-weight: bold;
}
h1 {
        color: #000000;
        font-size: 28pt;
}
h2 {
        border-bottom: 1px solid #CCCCCC;
        color: #000000;
        font-size: 24px;
}
h3 {
        font-size: 18px;
}
h4 {
        font-size: 16px;
}
h5 {
        font-size: 14px;
}
h6 {
        color: #777777;
        background-color: inherit;
        font-size: 14px;
}
hr {
        height: 0.2em;
        border: 0;
        color: #CCCCCC;
        background-color: #CCCCCC;
display: inherit;
}
p, blockquote, ul, ol, dl, li, table, pre {
        margin: 15px 0;
}
a, a:visited {
        color: #4183C4;
        background-color: inherit;
        text-decoration: none;
}
#message {
        border-radius: 6px;
        border: 1px solid #ccc;
        display:block;
        width:100%;
        height:60px;
        margin:6px 0px;
}
button, #ws {
        font-size: 12 pt;
        padding: 4px 6px;
        border-radius: 5px;
        border: 1px solid #bbb;
        background-color: #eee;
}
code, pre, #ws, #message {
        font-family: Monaco;
        font-size: 10pt;
        border-radius: 3px;
        background-color: #F8F8F8;
        color: inherit;
}
code {
        border: 1px solid #EAEAEA;
        margin: 0 2px;
        padding: 0 5px;
}
pre {
        border: 1px solid #CCCCCC;
        overflow: auto;
        padding: 4px 8px;
}
pre > code {
        border: 0;
        margin: 0;
        padding: 0;
}
#ws { background-color: #f8f8f8; }
.bloop_markdown table {
border-collapse: collapse;
font-family: Helvetica, arial, freesans, clean, sans-serif;
color: rgb(51, 51, 51);
font-size: 15px; line-height: 25px;
padding: 0; }
.bloop_markdown table tr {
border-top: 1px solid #cccccc;
background-color: white;
margin: 0;
padding: 0; }
.bloop_markdown table tr:nth-child(2n) {
background-color: #f8f8f8; }
.bloop_markdown table tr th {
font-weight: bold;
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr td {
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr th :first-child, table tr td :first-child {
margin-top: 0; }
.bloop_markdown table tr th :last-child, table tr td :last-child {
margin-bottom: 0; }
.bloop_markdown blockquote{
border-left: 4px solid #dddddd;
padding: 0 15px;
color: #777777; }
blockquote > :first-child {
margin-top: 0; }
blockquote > :last-child {
margin-bottom: 0; }
code, pre, #ws, #message {
word-break: normal;
word-wrap: normal;
}
hr {
display: inherit;
}
.bloop_markdown :first-child {
-webkit-margin-before: 0;
}
code, pre, #ws, #message {
font-family: Menlo, Consolas, Liberation Mono, Courier, monospace;
}
.send { color:#77bb77; }
.server { color:#7799bb; }
.error { color:#AA0000; }</style><div class="bloop_markdown"><p>Proposal is moved to a git repo: <a href="https://github.com/DevAndArtist/swift-evolution/blob/extensions_access_modifiers/proposals/nnnn-extensions-access-modifiers.md">https://github.com/DevAndArtist/swift-evolution/blob/extensions_access_modifiers/proposals/nnnn-extensions-access-modifiers.md</a></p>
<ul>
<li>I also updated a few things for readability.</li>
</ul>
<hr>
<h1 id="revisingaccessmodifiersonextensions">Revising access modifiers on extensions</h1>
<ul>
<li>Proposal: <a href="nnnn-extensions-access-modifiers.md">SE-NNNN</a></li>
<li>Author: <a href="https://github.com/DevAndArtist">Adrian Zubarev</a></li>
<li>Status: <strong><a href="#rationale">Awaiting review</a></strong></li>
<li>Review manager: TBD</li>
</ul>
<h2 id="introduction">Introduction</h2>
<p>One great goal for Swift 3 is to sort out any source breaking language changes. This proposal aims to fix access modifier inconsistency on extensions compared to other scope declarations types.</p>
<p>Swift-evolution thread: <a href="https://lists.swift.org/pipermail/swift-evolution/Week-of-Mon-20160620/022144.html">[Proposal] Revising access modifiers on extensions</a></p>
<h2 id="motivation">Motivation</h2>
<p>When declaring members on extensions which don’t have an explicit access modifier in Swift 2.2, it is possible to create an <strong>implicitly public extension</strong> by applying a <em>public</em> modifier to at least one extension member.</p>
<pre><code class="swift">public struct A { … }
// Implicitly public
extension A {
public var member1: SomeType { … }
// Implicitly internal
func member2() { … }
}
// Implicitly internal
extension A {
// Implicitly internal
var member3: SomeType { … }
}
</code></pre>
<p>Furthermore in Swift 2.2 it is not allowed to apply an <em>access modifier</em> on extensions when a <em>type inheritance clause</em> is present:</p>
<pre><code class="swift">public protocol B { … }
// 'public' modifier cannot be used with
// extensions that declare protocol conformances
public extension A : B { … }
</code></pre>
<h2 id="proposedsolution">Proposed solution</h2>
<ol>
<li><p>Allow access modifier on extensions when a type inheritance clause is present.</p></li>
<li><p>Remove the behavior of an implicit public extension.</p></li>
</ol>
<p>This changes should make access modifier on extensions consistent to classes, structs and enums (and <a href="https://github.com/apple/swift-evolution/blob/master/proposals/0025-scoped-access-level.md">SE–0025</a>).</p>
<h4 id="thecurrentgrammarwillnotchange:">The current grammar will not change:</h4>
<p><em>extension-declaration</em> → <em>access-level-modifier</em><sub>opt</sub> <strong>extension</strong> <em>type-identifier</em> <em>type-inheritance-clause</em><sub>opt</sub> <em>extension-body</em></p>
<p><em>extension-declaration</em> → <em>access-level-modifier</em><sub>opt</sub> <strong>extension</strong> <em>type-identifier</em> <em>requirement-clause</em> <em>extension-body</em></p>
<p><em>extension-body</em> → <strong>{</strong> <em>declarations</em><sub>opt</sub> <strong>}</strong></p>
<p>Iff the <em>access-level-modifier</em> is not present, the access modifier on extensions should always be implicitly <strong>internal</strong>.</p>
<h4 id="impactonapis:">Impact on APIs:</h4>
<p>Current version:</p>
<pre><code class="swift">/// Implementation version
///========================
public protocol Y {
func member()
}
public struct X { … }
// Implicitly public
extension X : Y {
public func member() { ... }
// Implicitly internal
func anotherMember() { ... }
}
/// Imported modele version
///========================
public protocol Y {
func member()
}
public struct X { ... }
// Missing `public` modifier
extension X : Y {
public func member() { ... }
}
</code></pre>
<p>New Version:</p>
<pre><code class="swift">/// Implementation version
///========================
public extension X : Y {
public func member() { ... }
// Implicitly internal
func anotherMember() { ... }
}
/// Imported modele version
///========================
public extension X : Y {
public func member() { ... }
}
</code></pre>
<h2 id="impactonexistingcode">Impact on existing code</h2>
<p>This is a source-breaking change that can be automated by a migrator, by simply scanning the <em>extension-body</em> for at least one <em>public</em> modifier on its members. Iff a <em>public</em> modifier was found on any member, the migrator can add an explicit <em>public</em> modifier to the extension itself.</p>
<h2 id="alternativesconsidered">Alternatives considered</h2>
<ul>
<li>No other alternative were considered for this proposal.</li>
</ul>
<h2 id="rationale">Rationale</h2>
<p>On [Date], the core team decided to <strong>(TBD)</strong> this proposal.
When the core team makes a decision regarding this proposal,
their rationale for the decision will be written here.</p>
<p></p></div><div class="bloop_original_html"><style>
body {
        font-family: "Helvetica Neue", Helvetica, Arial, sans-serif;
        padding:1em;
        margin:auto;
        background:#fefefe;
}
h1, h2, h3, h4, h5, h6 {
        font-weight: bold;
}
h1 {
        color: #000000;
        font-size: 28pt;
}
h2 {
        border-bottom: 1px solid #CCCCCC;
        color: #000000;
        font-size: 24px;
}
h3 {
        font-size: 18px;
}
h4 {
        font-size: 16px;
}
h5 {
        font-size: 14px;
}
h6 {
        color: #777777;
        background-color: inherit;
        font-size: 14px;
}
hr {
        height: 0.2em;
        border: 0;
        color: #CCCCCC;
        background-color: #CCCCCC;
display: inherit;
}
p, blockquote, ul, ol, dl, li, table, pre {
        margin: 15px 0;
}
a, a:visited {
        color: #4183C4;
        background-color: inherit;
        text-decoration: none;
}
#message {
        border-radius: 6px;
        border: 1px solid #ccc;
        display:block;
        width:100%;
        height:60px;
        margin:6px 0px;
}
button, #ws {
        font-size: 12 pt;
        padding: 4px 6px;
        border-radius: 5px;
        border: 1px solid #bbb;
        background-color: #eee;
}
code, pre, #ws, #message {
        font-family: Monaco;
        font-size: 10pt;
        border-radius: 3px;
        background-color: #F8F8F8;
        color: inherit;
}
code {
        border: 1px solid #EAEAEA;
        margin: 0 2px;
        padding: 0 5px;
}
pre {
        border: 1px solid #CCCCCC;
        overflow: auto;
        padding: 4px 8px;
}
pre > code {
        border: 0;
        margin: 0;
        padding: 0;
}
#ws { background-color: #f8f8f8; }
.bloop_markdown table {
border-collapse: collapse;
font-family: Helvetica, arial, freesans, clean, sans-serif;
color: rgb(51, 51, 51);
font-size: 15px; line-height: 25px;
padding: 0; }
.bloop_markdown table tr {
border-top: 1px solid #cccccc;
background-color: white;
margin: 0;
padding: 0; }
.bloop_markdown table tr:nth-child(2n) {
background-color: #f8f8f8; }
.bloop_markdown table tr th {
font-weight: bold;
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr td {
border: 1px solid #cccccc;
margin: 0;
padding: 6px 13px; }
.bloop_markdown table tr th :first-child, table tr td :first-child {
margin-top: 0; }
.bloop_markdown table tr th :last-child, table tr td :last-child {
margin-bottom: 0; }
.bloop_markdown blockquote{
border-left: 4px solid #dddddd;
padding: 0 15px;
color: #777777; }
blockquote > :first-child {
margin-top: 0; }
blockquote > :last-child {
margin-bottom: 0; }
code, pre, #ws, #message {
word-break: normal;
word-wrap: normal;
}
hr {
display: inherit;
}
.bloop_markdown :first-child {
-webkit-margin-before: 0;
}
code, pre, #ws, #message {
font-family: Menlo, Consolas, Liberation Mono, Courier, monospace;
}
.send { color:#77bb77; }
.server { color:#7799bb; }
.error { color:#AA0000; }</style><div class="bloop_markdown"><p><br></p></div><div class="bloop_original_html"><blockquote type="cite" class="clean_bq"><span><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div><div class="bloop_original_html"><blockquote type="cite" class="clean_bq"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div><div class="bloop_original_html"><div id="bloop_sign_1466841186407026176" class="bloop_sign">
</div>
</div>
<div class="bloop_markdown"></div>
</div>
</div>
</blockquote>
</div>
<div class="bloop_markdown"></div>
</div></div></span></blockquote></div><div class="bloop_markdown"><p></p></div><style>body{font-family:Helvetica,Arial;font-size:13px}</style></div><div class="bloop_markdown"><p></p></div></div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>swift-evolution mailing list</span><br><span><a href="mailto:swift-evolution@swift.org">swift-evolution@swift.org</a></span><br><span><a href="https://lists.swift.org/mailman/listinfo/swift-evolution">https://lists.swift.org/mailman/listinfo/swift-evolution</a></span><br></div></blockquote></div></body></html>