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

blue warnings in "SVGeocoder.m" and "SVGeocoder.m" #20

Closed
voicecho opened this issue Jun 6, 2012 · 1 comment
Closed

blue warnings in "SVGeocoder.m" and "SVGeocoder.m" #20

voicecho opened this issue Jun 6, 2012 · 1 comment

Comments

@voicecho
Copy link

voicecho commented Jun 6, 2012

I appreciate the "SVGeocoder" you have created. It works well except six blue warnings after analyzing.
Three blue warnings in "SVGeocoder.m":
1)Potential leak of an object allocated on line 135: self.request = ...
2)Potential leak of an object allocated on line 211: self.responseData = ...
3)Potential leak of an object allocated on line 214: self.connection = ...

And Three blue warnings in "JSONKit.m":
1)Dead code on line 744: anObject = [anObject retain]
2) Dead code on line 791: anObject = [anObject retain]
3)Logic error on line 865: encodeState->serializeOptionFlags = optionFlags

Xcode version is 4.2

ps. why not use NSJSONSerialization Class to to convert JSON to Foundation objects?

@samvermette
Copy link
Member

Just merged a pull request that fixes the SVGeocoder warnings. JSONKit isn't mine, I invite you to post these warnings on the JSONKit issues.

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

No branches or pull requests

2 participants