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

ARC Compatibility #1

Closed
wants to merge 26 commits into from
Closed

ARC Compatibility #1

wants to merge 26 commits into from

Conversation

jessedc
Copy link

@jessedc jessedc commented Dec 6, 2011

I've made the DCIntrospect project, and implementation file ARC compatible.

@sgleadow
Copy link
Owner

sgleadow commented Dec 7, 2011

Search and replace 'retain' with 'strong' and 'assign' with 'weak'?

@jessedc
Copy link
Author

jessedc commented Dec 7, 2011

I thought about it (even did it, but pulled it out) but I'm not sure what the effect would be if you used the static library in a project that wasn't using arc. (meryl for example)

Sent from my iPad

On 08/12/2011, at 9:04 AM, sgleadowreply@reply.github.com wrote:

Search and replace 'retain' with 'strong' and 'assign' with 'weak'?


Reply to this email directly or view it on GitHub:
#1 (comment)

jessedc and others added 20 commits December 23, 2011 11:34
This reverts commit 518b509, reversing
changes made to f178474.
Improved property logging where before only the object's class and
UIView class properties would be dumped, skipping any super classes
in between.
Couldn't see a reason to not log properties for these two classes when
all others are logged. The property output for these two classes is very
informative.
error: unknown pragma ignored [-Werror,-Wunknown-pragmas]
fixed really minor but confusing description for 1st user.
Updated logPropertiesForObject to log all superclass properties between object & UIView
@jessedc jessedc closed this Feb 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants