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

ObjcBridge fixes #141

Merged
merged 9 commits into from Aug 29, 2011
Merged

ObjcBridge fixes #141

merged 9 commits into from Aug 29, 2011

Conversation

mkroehnert
Copy link
Contributor

Hi,

here is a first set of fixes for the ObjcBridge.

The most notable on is that it is now again possible to create Objc classes from Io.

You might want to review if the change introducing the runtime function class_isMetaClass()
does not break anything you are working on currently.

mkroehnert added 9 commits August 21, 2011 14:39
…instancesRespondToSelector]

int this case ocject is a class and we want to
check if the instances respond to the selector
and not the class itself
…he old check with calls to the objc runtime
…jcBridge specific CHash functions weren't set on classProtos)
…_class_addMethod() to reflect the order of the runtime method class_addMethod()
stevedekorte added a commit that referenced this pull request Aug 29, 2011
@stevedekorte stevedekorte merged commit 3c4c41c into IoLanguage:master Aug 29, 2011
@stevedekorte
Copy link
Member

Looks great - will try it out. Thanks mkroehnert!

@stevedekorte
Copy link
Member

Undefined symbols for architecture x86_64:
"_CHash_setObjcBridgeHashFunctions", referenced from:
_IoObjcBridge_proto in IoObjcBridge.o
+[ObjcSubclass classProtos] in ObjcSubclass.o

@stevedekorte
Copy link
Member

Nevermind, that error was in IoLanguageKit - I shouldn't have expected you to update that :)

@mkroehnert
Copy link
Contributor Author

Ok, I didn't know of the LanguageKit project.
Maybe we could add a target to the CMake files so that it can be built without the Xcode project.

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

Successfully merging this pull request may close these issues.

None yet

2 participants