<html><body><div>Review link: <blockquote type="cite"><div class="msg-quote"><div class="_stretch"><span class="body-text-content"><a href="https://github.com/apple/swift-evolution/blob/master/proposals/0117-non-public-subclassable-by-default.md" data-mce-href="https://github.com/apple/swift-evolution/blob/master/proposals/0117-non-public-subclassable-by-default.md">https://github.com/apple/swift-evolution/blob/master/proposals/0117-non-public-subclassable-by-default.md</a><br></span></div></div></blockquote></div><div><br data-mce-bogus="1"></div><div><br data-mce-bogus="1"></div><div><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;">* What is your evaluation of the proposal?</span></div><div>+1 for the implementation. +0.5 for the concept. I think this is a clean interface for what open should be, and am glad at the simplification. This seems very "Swifty", and much better than the first proposal. I'm concerned that this may be over-limiting developers, and while everything looks great in theory, I am concerned that this may not be a good decision for real-world development. That said, I also think there are real wins in the ability to at a later date finalise a class for performance reasons, and provides clear structure for how a subclass is designed be used.</div><div><br style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"> * Is the problem being addressed significant enough to warrant a change to Swift?</span></div><div>I think that sorting out our inheritance story for public frameworks is important if we are going to start seeing public Swift Frameworks in the wild more often.</div><div><br style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"> * Does this proposal fit well with the feel and direction of Swift?</span></div><div>Yes.</div><div><br style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"> * If you have used other languages or libraries with a similar feature, how do you feel that this proposal compares to those?</span></div><div>I haven't. I've used Obj-C a lot. The ability to fix some issues with patching leads me to be concerned that we are overly restricting things here.</div><div><br style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;"> * How much effort did you put into your review? A glance, a quick reading, or an in-depth study?</span></div><div><span style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;" data-mce-style="font-family: SFNSText, 'Helvetica Neue', Helvetica, sans-serif; white-space: pre-wrap;">I've been following the threads since this discussion started months back, in discussions recently over it, and have read thoroughly both reviews.</span></div><div><br></div><div></div><div><blockquote type="cite"><div class="msg-quote"></div></blockquote></div></body></html>