Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use NSStringFromClass if the argument is of type "Class." #192

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

davidmurray
Copy link

No description provided.

@rpetrich
Copy link
Collaborator

Does NSStringFromClass initialize the class? Ideally %log() shouldn't initialize classes, and should handle nulls (as it does now)

@kirb
Copy link
Member

kirb commented Aug 14, 2016

This was brought up a while back, but I can't seem to find where the PR is…

CFLog already supports Class when used with %@. This isn't needed.

@rpetrich
Copy link
Collaborator

It does so via CFCopyDescription, which will initialize the class.

@kirb
Copy link
Member

kirb commented Aug 14, 2016

Ah, ick.

@DHowett
Copy link
Collaborator

DHowett commented Aug 14, 2016 via email

@DHowett
Copy link
Collaborator

DHowett commented Aug 14, 2016 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants