GTMOAuth2ViewControllerTouch needs refactoring about gtm-oauth2 HOT 1 CLOSED

Thanks for your feedback.

> Needs better MVC separation.

Considering there are two separate views and view controllers for the same 
underlying model, and there is some shared logic across platforms, the current 
model has held up reasonably well.

> There are two classes defined in the same class file 
GTMOAuth2ViewControllerTouch and GTMOAuth2Keychain.

Proliferation of source files in shared code makes build issues even more 
challenging. Colocating related classes here is intentional.

> Needs ARC compliance.

The code predates ARC, but it interoperates fine in an ARC application.

> The finishedSelector can be removed entirely as developers should use blocks 
instead.

Choice of blocks or callback selectors depends on the nature of the callback. 
Selectors are still superior where the callback is substantially complex and 
independent of the initiating code.

> Needs support for Storyboard UIs with segue to this authentication controller.

This project does predate Storyboards. It should be updated for those, and for 
modern UI generally.