Re: [swift-evolution] [Pitch] Rename Mirror

2016-08-10 Thread Anton Zhilin via swift-evolution
2016-08-11 0:40 GMT+03:00 Slava Pestov : > > I think pointing a Mirror at a metatype value could be a good a mechanism > for getting information about the type's stored properties and other > members. > > FWIW Mirrors are inspired by a feature of the same name in the Self >

Re: [swift-evolution] [Pitch] Rename Mirror

2016-08-10 Thread Slava Pestov via swift-evolution
> On Jul 21, 2016, at 4:06 PM, Anton Zhilin via swift-evolution > wrote: > > 2016-07-22 1:34 GMT+03:00 Dmitri Gribenko >: > > Mirror.DisplayStyle contains optional and set as special cases, but does not > > contain

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-31 Thread Thorsten Seitz via swift-evolution
> Am 22.07.2016 um 01:06 schrieb Anton Zhilin via swift-evolution > : > > 2016-07-22 1:34 GMT+03:00 Dmitri Gribenko : >> >> > Mirror.DisplayStyle contains optional and set as special cases, but does >> > not >> > contain function >> > Mirror

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-22 Thread Leonardo Pessoa via swift-evolution
I also don't think this is a worthy change. Mirror can be extended or even changed in the future to become a true reflection base class. No benefits added and breaking code are to me a reason to not go forward with this. L On 21 July 2016 at 22:05, Anton Zhilin via swift-evolution

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Anton Zhilin via swift-evolution
Somehow I did not send a copy to evolution the first time: I have to agree. Stopping the discussion on this until after Swift 3. And response from Dmitri: Thanks! Just wanted to be clear -- I am not against reflection. And I am concerned whether the current design of Mirror will work for the

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Dmitri Gribenko via swift-evolution
On Thu, Jul 21, 2016 at 4:49 PM, Anton Zhilin wrote: > 2016-07-22 2:39 GMT+03:00 Dmitri Gribenko : >> >> On Thu, Jul 21, 2016 at 4:06 PM, Anton Zhilin >> wrote: >> > Moreover, types can completely customize contents of their >>

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Anton Zhilin via swift-evolution
2016-07-22 2:39 GMT+03:00 Dmitri Gribenko : > On Thu, Jul 21, 2016 at 4:06 PM, Anton Zhilin > wrote: > > Moreover, types can completely customize contents of their > > 'Mirror's. This is incompatible with laziness and with how reflection > should > >

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Austin Zheng via swift-evolution
I agree with Dmitri. I would rather see a proper design for reflection (similar to Completing Generics) before we start making any changes to the existing machinery. Best, Austin On Thu, Jul 21, 2016 at 4:39 PM, Dmitri Gribenko via swift-evolution < swift-evolution@swift.org> wrote: > On Thu,

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Dmitri Gribenko via swift-evolution
On Thu, Jul 21, 2016 at 4:06 PM, Anton Zhilin wrote: > 2016-07-22 1:34 GMT+03:00 Dmitri Gribenko : >> >> > Mirror.DisplayStyle contains optional and set as special cases, but does >> > not >> > contain function >> > Mirror collects all information

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Anton Zhilin via swift-evolution
2016-07-22 1:34 GMT+03:00 Dmitri Gribenko : > > > Mirror.DisplayStyle contains optional and set as special cases, but does > not > > contain function > > Mirror collects all information possible at initialization, while for > true > > reflection we want laziness > > Mirror

Re: [swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Dmitri Gribenko via swift-evolution
On Thu, Jul 21, 2016 at 3:25 PM, Anton Zhilin via swift-evolution wrote: > https://github.com/Anton3/swift-evolution/blob/master/proposals/-rename-mirror.md > > Rename Mirror > > Proposal: SE- > Author: Anton Zhilin, Adrian Zubarev > Status: Awaiting review >

[swift-evolution] [Pitch] Rename Mirror

2016-07-21 Thread Anton Zhilin via swift-evolution
https://github.com/Anton3/swift-evolution/blob/master/proposals/-rename-mirror.md Rename Mirror - Proposal: SE- - Author: Anton Zhilin , Adrian Zubarev - Status: *Awaiting review* -