[Proposal] Move public AutoreleasingUnsafeMutablePointer API from StdlibCore -> Objective C Overlay

Michael_Gottesman · 2016-07-21 17:02:46 UTC

Hello everyone.

This is a proposal to move AutoreleasingUnsafeMutablePointer from StdlibCore to the Objective C overlay. The reason to do this is that ideally StdlibCore would not have any code specific to ObjC in it. This is a small step that we can take in this direction by moving the public API of AutoreleasingUnsafeMutablePointer to the ObjC overlay. The way that this would be done is that the implementation for AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but we would add an underscore to make that API private to the standard library. Then inside the ObjC overlay, we would create a typealias to the internal API with the non-underscored name (i.e. make the typealias the public API). Then at a later point in time we can move the implementation. The reason why I am not proposing to move the implementation today is that there are dependencies in Array/Hashed Collections on AutoreleasingUnsafeMutablePointer. Moving that code is a larger project that is preconditioned on this proposal going in first as an incremental step.

Thoughts, flames, etc?
Michael

gribozavr · 2016-07-21 17:07:33 UTC

+1.

Dmitri

···

On Thu, Jul 21, 2016 at 10:02 AM, Michael Gottesman via swift-evolution <swift-evolution@swift.org> wrote:

Hello everyone.

This is a proposal to move AutoreleasingUnsafeMutablePointer from StdlibCore to the Objective C overlay.

--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr@gmail.com>*/

dabrahams · 2016-07-21 17:32:26 UTC

Hello everyone.

This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in
it. This is a small step that we can take in this direction by moving
the public API of AutoreleasingUnsafeMutablePointer to the ObjC
overlay.

+1

To avoid confusion among reviewers, I feel I need to point out that
everything below this point is implementation detail that IMO has no
relevance to the evolution list.

The way that this would be done is that the implementation
for AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but
we would add an underscore to make that API private to the standard
library. Then inside the ObjC overlay, we would create a typealias to
the internal API with the non-underscored name (i.e. make the
typealias the public API). Then at a later point in time we can move
the implementation. The reason why I am not proposing to move the
implementation today is that there are dependencies in Array/Hashed
Collections on AutoreleasingUnsafeMutablePointer. Moving that code is
a larger project that is preconditioned on this proposal going in
first as an incremental step.

I am not confident that we will ever be able to move that code, but as I
said before, that's really irrelevant here. Let's do it!

···

on Thu Jul 21 2016, Michael Gottesman <swift-evolution@swift.org> wrote:

--
Dave

Joe_Groff · 2016-07-21 18:19:12 UTC

Sounds awesome. Someone will need to ping Bret Victor to update his T-shirt: https://twitter.com/worrydream/status/558533126444445697

-Joe

···

On Jul 21, 2016, at 10:02 AM, Michael Gottesman via swift-evolution <swift-evolution@swift.org> wrote:

Hello everyone.

This is a proposal to move AutoreleasingUnsafeMutablePointer from StdlibCore to the Objective C overlay. The reason to do this is that ideally StdlibCore would not have any code specific to ObjC in it. This is a small step that we can take in this direction by moving the public API of AutoreleasingUnsafeMutablePointer to the ObjC overlay. The way that this would be done is that the implementation for AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but we would add an underscore to make that API private to the standard library. Then inside the ObjC overlay, we would create a typealias to the internal API with the non-underscored name (i.e. make the typealias the public API). Then at a later point in time we can move the implementation. The reason why I am not proposing to move the implementation today is that there are dependencies in Array/Hashed Collections on AutoreleasingUnsafeMutablePointer. Moving that code is a larger project that is preconditioned on this proposal going in first as an incremental step.

Thoughts, flames, etc?

Felix_Cloutier1 · 2016-07-22 15:23:20 UTC

I agree on principle for moving it out of stdlib, but what's the Objective-C overlay?

Félix

···

Le 21 juil. 2016 à 10:02:46, Michael Gottesman via swift-evolution <swift-evolution@swift.org> a écrit :

Hello everyone.

This is a proposal to move AutoreleasingUnsafeMutablePointer from StdlibCore to the Objective C overlay. The reason to do this is that ideally StdlibCore would not have any code specific to ObjC in it. This is a small step that we can take in this direction by moving the public API of AutoreleasingUnsafeMutablePointer to the ObjC overlay. The way that this would be done is that the implementation for AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but we would add an underscore to make that API private to the standard library. Then inside the ObjC overlay, we would create a typealias to the internal API with the non-underscored name (i.e. make the typealias the public API). Then at a later point in time we can move the implementation. The reason why I am not proposing to move the implementation today is that there are dependencies in Array/Hashed Collections on AutoreleasingUnsafeMutablePointer. Moving that code is a larger project that is preconditioned on this proposal going in first as an incremental step.

Thoughts, flames, etc?
Michael
_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

benrimmington · 2016-07-22 18:05:11 UTC

If you want to move APIs which aren't cross-platform out of stdlib, should there be a similar proposal for the PlaygroundQuickLook enum and CustomPlaygroundQuickLookable protocol?

<https://github.com/apple/swift/blob/master/stdlib/public/core/Mirror.swift>

<https://github.com/apple/swift-xcode-playground-support>

-- Ben

···

On 21 Jul 2016, at 18:02, Michael Gottesman wrote:

This is a proposal to move AutoreleasingUnsafeMutablePointer from StdlibCore to the Objective C overlay. The reason to do this is that ideally StdlibCore would not have any code specific to ObjC in it. This is a small step that we can take in this direction by moving the public API of AutoreleasingUnsafeMutablePointer to the ObjC overlay. The way that this would be done is that the implementation for AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but we would add an underscore to make that API private to the standard library. Then inside the ObjC overlay, we would create a typealias to the internal API with the non-underscored name (i.e. make the typealias the public API). Then at a later point in time we can move the implementation. The reason why I am not proposing to move the implementation today is that there are dependencies in Array/Hashed Collections on AutoreleasingUnsafeMutablePointer. Moving that code is a larger project that is preconditioned on this proposal going in first as an incremental step.

Austin · 2016-07-21 20:33:41 UTC

This proposal is great.

Complaining about an API that 99% of users won't ever have to touch and
exists to facilitate NS/CF interop is a bit precious, to be charitable.

···

On Thu, Jul 21, 2016 at 11:19 AM, Joe Groff via swift-evolution < swift-evolution@swift.org> wrote:

> On Jul 21, 2016, at 10:02 AM, Michael Gottesman via swift-evolution < > swift-evolution@swift.org> wrote:
>
> Hello everyone.
>
> This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in it. This is
a small step that we can take in this direction by moving the public API of
AutoreleasingUnsafeMutablePointer to the ObjC overlay. The way that this
would be done is that the implementation for
AutoreleasingUnsafeMutablePointer would remain in StdlibCore, but we would
add an underscore to make that API private to the standard library. Then
inside the ObjC overlay, we would create a typealias to the internal API
with the non-underscored name (i.e. make the typealias the public API).
Then at a later point in time we can move the implementation. The reason
why I am not proposing to move the implementation today is that there are
dependencies in Array/Hashed Collections on
AutoreleasingUnsafeMutablePointer. Moving that code is a larger project
that is preconditioned on this proposal going in first as an incremental
step.
>
> Thoughts, flames, etc?

Sounds awesome. Someone will need to ping Bret Victor to update his
T-shirt: https://twitter.com/worrydream/status/558533126444445697

-Joe

_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

gribozavr · 2016-07-22 15:32:45 UTC

It is the overlay for the 'ObjectiveC' module where NSObject is defined.

Dmitri

···

On Fri, Jul 22, 2016 at 8:23 AM, Félix Cloutier <swift-evolution@swift.org> wrote:

I agree on principle for moving it out of stdlib, but what's the Objective-C
overlay?

--
main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
(j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr@gmail.com>*/

dabrahams · 2016-07-22 18:42:50 UTC

That is a good question! We don't have an obvious existing module into
which to move them, though, do we?

···

on Fri Jul 22 2016, Ben Rimmington <swift-evolution@swift.org> wrote:

On 21 Jul 2016, at 18:02, Michael Gottesman wrote:

This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in

it. This is a small step that we can take in this direction by
moving the public API of AutoreleasingUnsafeMutablePointer to the
ObjC overlay. The way that this would be done is that the
implementation for AutoreleasingUnsafeMutablePointer would remain in
StdlibCore, but we would add an underscore to make that API private
to the standard library. Then inside the ObjC overlay, we would
create a typealias to the internal API with the non-underscored name
(i.e. make the typealias the public API). Then at a later point in
time we can move the implementation. The reason why I am not
proposing to move the implementation today is that there are
dependencies in Array/Hashed Collections on
AutoreleasingUnsafeMutablePointer. Moving that code is a larger
project that is preconditioned on this proposal going in first as an
incremental step.

If you want to move APIs which aren't cross-platform out of stdlib,
should there be a similar proposal for the PlaygroundQuickLook enum
and CustomPlaygroundQuickLookable protocol?

<https://github.com/apple/swift/blob/master/stdlib/public/core/Mirror.swift>

<https://github.com/apple/swift-xcode-playground-support>

--
Dave

Michael_Gottesman · 2016-07-22 19:21:59 UTC

This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in

it. This is a small step that we can take in this direction by
moving the public API of AutoreleasingUnsafeMutablePointer to the
ObjC overlay. The way that this would be done is that the
implementation for AutoreleasingUnsafeMutablePointer would remain in
StdlibCore, but we would add an underscore to make that API private
to the standard library. Then inside the ObjC overlay, we would
create a typealias to the internal API with the non-underscored name
(i.e. make the typealias the public API). Then at a later point in
time we can move the implementation. The reason why I am not
proposing to move the implementation today is that there are
dependencies in Array/Hashed Collections on
AutoreleasingUnsafeMutablePointer. Moving that code is a larger
project that is preconditioned on this proposal going in first as an
incremental step.

If you want to move APIs which aren't cross-platform out of stdlib,
should there be a similar proposal for the PlaygroundQuickLook enum
and CustomPlaygroundQuickLookable protocol?

<https://github.com/apple/swift/blob/master/stdlib/public/core/Mirror.swift>

<https://github.com/apple/swift-xcode-playground-support>

That is a good question! We don't have an obvious existing module into
which to move them, though, do we?

This is just an incremental proposal. The playground work should be in a different proposal so we keep this proposal focused.

Michael

···

On Jul 22, 2016, at 11:42 AM, Dave Abrahams via swift-evolution <swift-evolution@swift.org> wrote:
on Fri Jul 22 2016, Ben Rimmington <swift-evolution@swift.org <mailto:swift-evolution@swift.org>> wrote:

On 21 Jul 2016, at 18:02, Michael Gottesman wrote:

--
Dave

_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org <mailto:swift-evolution@swift.org>
https://lists.swift.org/mailman/listinfo/swift-evolution

Erica_Sadun · 2016-07-22 19:32:16 UTC

This philosophy appeals to me, especially when considering Swift as a
cross-platform language. Would you all be open to "new module" proposals?
Would that be internal or through Swift Evolution? I'm thinking also of
the discussion about creating a Swift native Math module w/ JordanR and
Steve C for things that don't belong in the stdlib but would be nice to have around,
sanctioned native modules.

As for PlaygroundSupport, how does that tie into stdlib? I thought it was its own
thing, separate from stdlib. Wouldn't both quick look and lookable belong there?

-- E

···

On Jul 22, 2016, at 12:42 PM, Dave Abrahams via swift-evolution <swift-evolution@swift.org> wrote:

on Fri Jul 22 2016, Ben Rimmington <swift-evolution@swift.org <mailto:swift-evolution@swift.org>> wrote:

On 21 Jul 2016, at 18:02, Michael Gottesman wrote:

This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in

it. This is a small step that we can take in this direction by
moving the public API of AutoreleasingUnsafeMutablePointer to the
ObjC overlay. The way that this would be done is that the
implementation for AutoreleasingUnsafeMutablePointer would remain in
StdlibCore, but we would add an underscore to make that API private
to the standard library. Then inside the ObjC overlay, we would
create a typealias to the internal API with the non-underscored name
(i.e. make the typealias the public API). Then at a later point in
time we can move the implementation. The reason why I am not
proposing to move the implementation today is that there are
dependencies in Array/Hashed Collections on
AutoreleasingUnsafeMutablePointer. Moving that code is a larger
project that is preconditioned on this proposal going in first as an
incremental step.

If you want to move APIs which aren't cross-platform out of stdlib,
should there be a similar proposal for the PlaygroundQuickLook enum
and CustomPlaygroundQuickLookable protocol?

<https://github.com/apple/swift/blob/master/stdlib/public/core/Mirror.swift>

<https://github.com/apple/swift-xcode-playground-support>

That is a good question! We don't have an obvious existing module into
which to move them, though, do we?

dabrahams · 2016-07-22 22:06:48 UTC

This is a proposal to move AutoreleasingUnsafeMutablePointer from
StdlibCore to the Objective C overlay. The reason to do this is that
ideally StdlibCore would not have any code specific to ObjC in

it. This is a small step that we can take in this direction by
moving the public API of AutoreleasingUnsafeMutablePointer to the
ObjC overlay. The way that this would be done is that the
implementation for AutoreleasingUnsafeMutablePointer would remain in
StdlibCore, but we would add an underscore to make that API private
to the standard library. Then inside the ObjC overlay, we would
create a typealias to the internal API with the non-underscored name
(i.e. make the typealias the public API). Then at a later point in
time we can move the implementation. The reason why I am not
proposing to move the implementation today is that there are
dependencies in Array/Hashed Collections on
AutoreleasingUnsafeMutablePointer. Moving that code is a larger
project that is preconditioned on this proposal going in first as an
incremental step.

If you want to move APIs which aren't cross-platform out of stdlib,
should there be a similar proposal for the PlaygroundQuickLook enum
and CustomPlaygroundQuickLookable protocol?

<https://github.com/apple/swift/blob/master/stdlib/public/core/Mirror.swift>

<https://github.com/apple/swift-xcode-playground-support>

That is a good question! We don't have an obvious existing module into
which to move them, though, do we?

This is just an incremental proposal. The playground work should be in
a different proposal so we keep this proposal focused.

+1

···

on Fri Jul 22 2016, Michael Gottesman <swift-evolution@swift.org> wrote:

On Jul 22, 2016, at 11:42 AM, Dave Abrahams via swift-evolution >> <swift-evolution@swift.org> wrote:
on Fri Jul 22 2016, Ben Rimmington > >> <swift-evolution@swift.org >> <mailto:swift-evolution@swift.org>> >> wrote:

On 21 Jul 2016, at 18:02, Michael Gottesman wrote:

Michael

--
Dave

_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org
<mailto:swift-evolution@swift.org>
https://lists.swift.org/mailman/listinfo/swift-evolution
<https://lists.swift.org/mailman/listinfo/swift-evolution>

_______________________________________________
swift-evolution mailing list
swift-evolution@swift.org
https://lists.swift.org/mailman/listinfo/swift-evolution

--
Dave