Upgraded project files to the latest xcode and removed all warning #785

Closed
wants to merge 5 commits into
from

Projects

None yet

4 participants

@yosit
yosit commented Mar 13, 2012

three20 now compiles with no warning.

yosit added some commits Mar 13, 2012
@yosit yosit updated project files to Xcode 3.2 d06aaa1
@yosit yosit Fixed all warnings e78cdf2
@yosit yosit Merge branch 'master' of git://github.com/facebook/three20
Conflicts:
	samples/TTCatalog/Classes/TableDragRefreshController.m
	samples/TTCatalog/TTCatalog.xcodeproj/project.pbxproj
	src/Three20/Three20.xcodeproj/project.pbxproj
	src/Three20Core/Three20Core.xcodeproj/project.pbxproj
	src/Three20Network/Three20Network.xcodeproj/project.pbxproj
	src/Three20Style/Three20Style.xcodeproj/project.pbxproj
	src/Three20UI/Sources/TTModelViewController.m
	src/Three20UI/Three20UI.xcodeproj/project.pbxproj
	src/Three20UICommon/Three20UICommon.xcodeproj/project.pbxproj
	src/Three20UINavigator/Three20UINavigator.xcodeproj/project.pbxproj
474cffe
@diederich

Hm, this should actually be an implicit 'upcast'. If this gives a warning, maybe the include is missing?

Contributor

Looking at the diff to development branch (https://github.com/facebook/three20/pull/785/files) this shows only a single file change + the project changes. See my comment on the file change.

Regarding the Xcode update it looks to me that only LastUpgradeCheck was bumped. Nothing else to change? (IIRC Xcode asked me to switch to LLDB for pretty much all projects)

CVertex commented Apr 4, 2012

I'd like this merged in

Contributor

Facebook has not maintained or supported Three20 for some time, and we are closing its old and outstanding pull requests.

Many, many thanks for your support of the project. If you have any further questions, please don't hesitate to let me know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment