[swift-evolution] [Guidelines, First Argument Labels]: Prepositions inside the parens
Xiaodi Wu
xiaodi.wu at gmail.com
Thu Feb 11 13:57:58 CST 2016
Huge improvement overall moving the 'of'. However, these and related
functions stand out as regressing in clarity:
'''
- class func userFont(ofSize fontSize: CGFloat) -> NSFont?
- class func userFixedPitchFont(ofSize fontSize: CGFloat) -> NSFont?
+ class func userFontOf(size fontSize: CGFloat) -> NSFont?
+ class func userFixedPitchFontOf(size fontSize: CGFloat) -> NSFont?
'''
My rationalization: here, 'of' is being used differently than in other
places. In fact, userFont(ofSize:) could be equally renamed
userFont(withSize:) or userFont(havingSize:).
On Thu, Feb 11, 2016 at 1:35 PM, Douglas Gregor via swift-evolution
<swift-evolution at swift.org> wrote:
>
>> On Feb 11, 2016, at 10:53 AM, Dave Abrahams via swift-evolution <swift-evolution at swift.org> wrote:
>>
>>
>> on Thu Feb 11 2016, Charles Kissinger <swift-evolution at swift.org> wrote:
>>
>>> FWIW, I think these new guidelines strike a very nice balance between
>>> all of the competing concerns. The result of applying them to some of
>>> my own code could best be described as “different but not unpleasant”
>>> ;-)
>>>
>>> In looking over the diffs, one nice result I didn’t expect is that
>>> moving the prepositional phrases into the first argument label seems
>>> to increase the speed with which I absorb the general intent of a
>>> function -- the “at a glance” understanding -- by moving a “detail”
>>> (important as it might be) into the parameter list. (The split
>>> prepositional phrases had the opposite effect for me.)
>>
>> That's nice! What do you think about moving “of” back out of the
>> parens? Does that help or hurt?
>
>
> And lo!, a diff appeared!
>
> https://github.com/apple/swift-3-api-guidelines-review/commit/3f965346f1edea804a9bc6223d2824f43687e92f
>
> - Doug
>
> _______________________________________________
> swift-evolution mailing list
> swift-evolution at swift.org
> https://lists.swift.org/mailman/listinfo/swift-evolution
More information about the swift-evolution
mailing list