krzysztofzablocki / LinkedConsole

Clickable links in your Xcode console, so you never wonder which class logged the message.
http://merowing.info
MIT License
931 stars 63 forks source link

Nitpicks #2

Closed samsonjs closed 8 years ago

samsonjs commented 8 years ago

Not sure if you'll like all of these. You don't need Selector("foo") because it's StringLiteralConvertible but it's less explicit, so kind of a matter of taste there.

Same for changing the guard let to if let, but in that case it's simpler because you skip the additional check, and getViewByClassName always returns the correct class so it was effectively a nil check anyway.