Skip to content
This repository has been archived by the owner on Sep 9, 2020. It is now read-only.

Collect useful commits from other forks and package it in a pull request. #11

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

schmidp
Copy link

@schmidp schmidp commented Sep 10, 2013

Hi,

I went through a few forks that have reasonable commits which are useful for everyone and merged them in my fork.
This pull request should merge changes of the following repos:

https://github.com/baldmountain/Cocoa-Touch-Barcodes/ by @baldmountain
https://github.com/baz/Cocoa-Touch-Barcodes/ by @baz
https://github.com/netshade/Cocoa-Touch-Barcodes/
https://github.com/openresearch/Cocoa-Touch-Barcodes/

Tell me what you think about it.

@@ -23,7 +23,6 @@ TODO:
* Better documentation
* Really, the entire API pretty much needs a refactor. The UIImage category as entry point into generating images as a start, NKDBarcodeOffscreenView doesn't really need to subclass UIView, etc..
* Leak checks
* ARC compatibility? I guess?
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lol

@netshade
Copy link
Owner

Lot of good changes in here, I'll check out the branch this week and try it out - great idea to pull all these together, thanks.

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

Successfully merging this pull request may close these issues.

4 participants