[swift-evolution] [Review] SE-0005 Better Translation of Objective-C APIs Into Swift

Douglas Gregor dgregor at apple.com
Mon Feb 1 14:10:25 CST 2016


> On Jan 29, 2016, at 12:07 PM, Jordan Rose via swift-evolution <swift-evolution at swift.org> wrote:
> 
> 
>> On Jan 28, 2016, at 14:15, Dave Abrahams via swift-evolution <swift-evolution at swift.org <mailto:swift-evolution at swift.org>> wrote:
>> 
>>> I'm very used to "fooWithBar: baz" meaning either "get me the foo that
>>> has a bar matching baz" or "create me a foo with its bar set to
>>> baz". 
>> 
>> That's great, when that's what "with" means.
>> 
>>> But I'm not sure this new convention is any worse, now that the base
>>> name isn't assumed to include the first argument.
>> 
>> The problem is that, I'm guessing at least 50% of the time, "with" is
>> just used as a vacuous connector to make the method name sound
>> grammatical, and "fooWithBar" doesn't actually mean the "foo" has-a
>> "bar."  In these cases, it's actively misleading. I know that's not what
>> you were posting about, but I felt it had to be said :-/
> 
> I actually don't think this is true when "foo" is a noun (and searching through the selector dump Doug made a long time ago seems to back that up).
> 
> Exceptions:
> - "fooWithOptions", but that's already caught by the default parameter rule.
> - "fooWithLocale", which uses the locale to build the result.
> - "commonPrefixWithString", where the "with" isn't quite vacuous.
> 
> But when "foo" is a verb (or when it's a later parameter that's just "withBar") it does seem pretty vacuous.

This is a great observation. Pull request here shows what this does:

	https://github.com/apple/swift-3-api-guidelines-review/pull/9

and, from the cases we’ve looked at, does a fantastic job of distinguishing the cases where “with” is a separator vs. “with” meaning some kind of selection criteria.

	- Doug


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.swift.org/pipermail/swift-evolution/attachments/20160201/e71a7c14/attachment.html>


More information about the swift-evolution mailing list