> On Nov 16, 2017, at 2:23 PM, Ted Kremenek via swift-evolution 
> <swift-evolution@swift.org> wrote:
> 
> When reviewing a proposal, here are some questions to consider:
> 
> What is your evaluation of the proposal?
> 
I think the functionality is good, but I would like to see some thought on what 
future values could be to see if this is the best name/structure.

If it is just going to be the concept of a simulator, then something like 
isSimulated() might be better.  In the current form, I think we should have 
both ‘Simulator’ and ‘Device’ as options.

I would also like to see something shorter than targetEnvironment(), but it is 
somewhat infrequently used, so it isn’t that big a deal.  It is just compared 
to os() and arch(), this is kind of a beast.  It is a power user thing, so 
maybe something like ‘env()’ would work?  I normally try to avoid 
abbreviations, but we have arch() as precedent.  The word ‘Simulator’ should be 
what stands out...

Would Testing be a possible future environment?

> Is the problem being addressed significant enough to warrant a change to 
> Swift?
> 
Yes, Definitely!

> Does this proposal fit well with the feel and direction of Swift?
> 
Yes.
> If you have used other languages or libraries with a similar feature, how do 
> you feel that this proposal compares to those?
> 
I guess Objective C had something like this as well, which was more powerful, 
but also more messy.
> How much effort did you put into your review? A glance, a quick reading, or 
> an in-depth study?
> 
Quick Read

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

Reply via email to