<div dir="ltr">Ah, great, that's even better. My solution is overkill compared to this and I think you're covering all the interesting cases anyway. I would love to work on the implementation if it's not assigned to anyone internally yet. (I've seen you mentioned there's a radar already?)<div><br></div><div>Cheers.</div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><br>--<br>Slds,<br><br>Gonzalo.</div></div>
<br><div class="gmail_quote">On Thu, Jun 29, 2017 at 2:42 PM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br><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"><div>Thanks for following through on this, Gonzalo! (The two of us talked about this briefly at WWDC.) My idea was a little less clever than yours: just always include the internal members in the main header <i>unless</i> you provide the name of a second header to put them in. (I called this flag -emit-objc-internal-header-<wbr>path.) That's not quite as flexible, but does remove all the guesswork, at least for generated headers.</div><div><br></div><div>I filed a bug to track this work: <a href="https://bugs.swift.org/browse/SR-5221" target="_blank">SR-5221</a>.</div><div><br></div><div>Jordan</div><br><div><br><blockquote type="cite"><div><div class="h5"><div>On Jun 29, 2017, at 06:35, Gonzalo Larralde via swift-dev <<a href="mailto:swift-dev@swift.org" target="_blank">swift-dev@swift.org</a>> wrote:</div><br class="m_3620898678313281180Apple-interchange-newline"></div></div><div><div><div class="h5"><div dir="ltr"><div>I wanted to bring a few points into attention to discuss potential improvements on Mixed Frameworks support. The current status for Mixed Frameworks present some challenges, one specifically being how the Swift compiler generates the ObjC Interface Header. <br></div><div><br></div><div>At this moment, there's a detection procedure based on the presence of an entry point definition (either using the main file argument, or @UIApplicationMain). The ObjC Interface Header writer (swift/lib/PrintAsObjC/<wbr>PrintAsObjC.cpp) is using this detection to determine if the minimum accessibility level to be exported is either `internal` (for app targets) or `public` (for framework targets). This can be observed here: <a href="https://github.com/apple/swift/blob/master/lib/FrontendTool/FrontendTool.cpp#L652-L657" target="_blank">https://github.com/apple/<wbr>swift/blob/master/lib/<wbr>FrontendTool/FrontendTool.cpp#<wbr>L652-L657</a></div><div><br></div><div>The result of this is a difference on how default visibility is exposed to ObjC code in the same compilation scope.</div><div><br></div><div>Having this initial piece of code:</div><div><br></div><div>```</div><div>public class Test: NSObject {</div><div><span style="white-space:pre-wrap"> </span>public foo() {}</div><div><span style="white-space:pre-wrap"> </span>func bar() {}</div><div>}</div><div>```</div><div><br></div><div>results on the following Interface Header for Main App targets</div><div><br></div><div><target-name>-swift.h</div><div>```</div><div>@interface Test : NSObject</div><div>- (void)foo;</div><div>- (void)bar;</div><div>@end</div><div>```</div><div><br></div><div>and the following for Frameworks</div><div><br></div><div><target-name>-swift.h</div><div>```</div><div>@interface Test : NSObject</div><div>- (void)foo;</div><div>@end</div><div>```</div><div><br></div><div>This is clearly correct for the publicly visible interface of the framework, but not quite the expected result for the ObjC code compiled in the same target. In that scenario it would make more sense to me that all the `internal` methods are visible.</div><div><br></div><div>A potential solution for this problem is to generate two Interface Headers, one that exports all the `public` and `open` entities and members, and an additional header exposing internal entities and declaring categories to public entities and exposing internal members.</div><div><br></div><div>Something like:</div><div><br></div><div><target-name>-swift-internal.h</div><div>```</div><div>@interface Test (InternalMembers)</div><div>- (void)bar;</div><div>@end</div><div>```</div><div><br></div><div>After that it'll be Xcode's responsability to create both files, and mark them as Public and Project in the Headers Build Phase.</div><div><br></div><div>An initial implementation that I think would make sense in case this solution is accepted could be modifying the signature of `swift::printAsObjC` to require the list of access levels to export as a bitmask instead of just getting the minimum. That will enable the exporter to create the following set of files:</div><div><br></div><div>* Internal, Public, Open > <target-name>-swift.h for app targets</div><div>* Public, Open > <target-name>-swift.h for framework targets</div><div>* Internal > <target-name>-swift-internal.h for the internal entities and members on framework targets.</div><div><br></div><div>To make this happen a new argument needs to be passed to the compiler call. When it's not passed the default behaviour would remain the same, but when it is the behaviour needs to be explicitly defined. One option is to just get the explicit list of access levels to export (something like `-export=internal,public,open`<wbr>) or export levels to be defined (0 for app targets, 1 for public framework targets and 2 for the internal header framework targets for example)</div><div><br></div><div>I hope this feature proposal is complete enough to properly define the scope and discuss the viability of a possible solution. Otherwise please let me know.</div><div><br></div><div>Thanks.</div><div><div class="m_3620898678313281180gmail_signature"><br>--<br>Slds,<br><br>Gonzalo.</div></div>
</div></div></div>
______________________________<wbr>_________________<br>swift-dev mailing list<br><a href="mailto:swift-dev@swift.org" target="_blank">swift-dev@swift.org</a><br><a href="https://lists.swift.org/mailman/listinfo/swift-dev" target="_blank">https://lists.swift.org/<wbr>mailman/listinfo/swift-dev</a><br></div></blockquote></div><br></div></blockquote></div><br></div>