non ARC compliant zip classes

[krisbulman]

The dev-arc branch compiles fine with the -fno-obj-arc compile flag set on the following files

FileInZipInfo.m
ZipFile.m
ZipReadStream.m
ZipWriteStream.m

However, I was wondering what the background is here, does a standard arc migration on these files (removal of release, dealloc, etc) continue to cause problems after the fact?

[incanus]

Hadn't tried them yet -- just took a stab at ARC for the main library, which is why it's in a branch.

[incanus]

Just to follow up on this, I probably will leave it as non-ARC since it'a a dependency and outside code. I will be checking dependencies shortly, though, to update them if necessary.

[ronak2121]

There is an ARC-ed branch available in the Objective-Zip Submodule. Why not just switch to that branch? I'm trying to add -fno-obj-arc to the above files but I still get compilation errors complaining about ARC.

[incanus]

Cool, I don't believe this was around before. Will investigate.

[incanus]

@ronak2121 where you are seeing this ARC branch? I see an attempt at ARC that was later reverted in https://github.com/AgileBits/objective-zip

[ronak2121]

I see it here:

https://github.com/CoolEmber/Objective-Zip

---

From: Justin R. Miller notifications@github.com
To: mapbox/Simple-KML Simple-KML@noreply.github.com
Cc: ronak2121 ronak2121@yahoo.com
Sent: Wednesday, January 23, 2013 6:19 PM
Subject: Re: [Simple-KML] non ARC compliant zip classes (#12)

@ronak2121 where you are seeing this ARC branch? I see an attempt at ARC that was later reverted in https://github.com/AgileBits/objective-zip
—
Reply to this email directly or view it on GitHub.