[swift-evolution] [Review #2] SE-0117: Default classes to be non-subclassable publicly

John McCall rjmccall at apple.com
Sat Jul 16 12:54:56 CDT 2016


> On Jul 16, 2016, at 12:59 AM, Xiaodi Wu via swift-evolution <swift-evolution at swift.org> wrote:
> 
> 
>         * What is your evaluation of the proposal?
> 
> This is improved from the previous iteration. The code example needs updating, as both instances of `open func bar()` should be `public open func bar()` as outlined in the Proposed Design section.

Good catch.  I'll fix this.
 
>         * Does this proposal fit well with the feel and direction of Swift?
> 
> Yes, mostly. There is one comment in the code example that describes a restriction which does not fit with the direction of Swift. It is not the main focus of the proposal but I think should be changed. Namely, the proposal comments:
> 
> "[The declaration `[public] open func bar()` inside a class not marked `open`] raises a compilation error: a method can't be marked `open` if the class it belongs to can't be subclassed."
> 
> This is discordant with the direction resolved by the core team in the SE-0025 revisions, where it was stated with regard to access modifiers:
> 
> "The compiler should not warn when a broader level of access control is used within a type with more restrictive access, such as `internal` within a `private` type. This allows the owner of the type to design the access they would use were they to make the type more widely accessible."
> 
> Applying the same rationale here would suggest that the compiler should not raise an error if a method is marked `open` inside a non-`open` type, in order to allow the owner of the type to design as though to make it subclassable without actually having to do so.

That's true.  We'll consider this.

John.

> 
>         * If you have used other languages or libraries with a similar feature, how do you feel that this proposal compares to those?
> 
> Yes, I've used OOP in other languages. As discussed, this approach is different from that taken by many of those but is a deliberate step.
>  
>         * How much effort did you put into your review? A glance, a quick reading, or an in-depth study?
> 
> Followed the discussion, read proposal carefully.
>  
> 
> More information about the Swift evolution process is available at
> 
>         https://github.com/apple/swift-evolution/blob/master/process.md <https://github.com/apple/swift-evolution/blob/master/process.md>
> 
> Thank you,
> 
> -Chris Lattner
> Review Manager
> 
> 
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org <mailto:swift-evolution at swift.org>
> https://lists.swift.org/mailman/listinfo/swift-evolution <https://lists.swift.org/mailman/listinfo/swift-evolution>
> 
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org
> https://lists.swift.org/mailman/listinfo/swift-evolution

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160716/221bfba5/attachment.html>


More information about the swift-evolution mailing list