diff options
author | nicola <nicola@138bc75d-0d04-0410-961f-82ee72b054a4> | 2010-12-21 22:49:37 +0000 |
---|---|---|
committer | nicola <nicola@138bc75d-0d04-0410-961f-82ee72b054a4> | 2010-12-21 22:49:37 +0000 |
commit | 7d026d14fe10ddd8779c2b171097c4e8b4cdf9ea (patch) | |
tree | 7b230e110e8855f7e0345ca7c88cdfb6276368f2 /libobjc | |
parent | ce31fa3fe8776af79a61af8858b7d5bb14f22bd6 (diff) | |
download | gcc-7d026d14fe10ddd8779c2b171097c4e8b4cdf9ea.tar.gz |
In libobjc/:
2010-12-21 Nicola Pero <nicola.pero@meta-innovation.com>
PR libobjc/18764
* class.c (__objc_add_class_to_hash): Return YES if the class was
added, and NO if it already existed.
* init.c (__objc_init_class): If __objc_add_class_to_hash returns
NO, then abort the program with an error message.
* objc-private/runtime.h (__objc_add_class_to_hash): Updated
declaration.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@168139 138bc75d-0d04-0410-961f-82ee72b054a4
Diffstat (limited to 'libobjc')
-rw-r--r-- | libobjc/ChangeLog | 10 | ||||
-rw-r--r-- | libobjc/class.c | 28 | ||||
-rw-r--r-- | libobjc/init.c | 36 | ||||
-rw-r--r-- | libobjc/objc-private/runtime.h | 16 |
4 files changed, 56 insertions, 34 deletions
diff --git a/libobjc/ChangeLog b/libobjc/ChangeLog index 04243d4e504..9d6806f2ab0 100644 --- a/libobjc/ChangeLog +++ b/libobjc/ChangeLog @@ -1,5 +1,15 @@ 2010-12-21 Nicola Pero <nicola.pero@meta-innovation.com> + PR libobjc/18764 + * class.c (__objc_add_class_to_hash): Return YES if the class was + added, and NO if it already existed. + * init.c (__objc_init_class): If __objc_add_class_to_hash returns + NO, then abort the program with an error message. + * objc-private/runtime.h (__objc_add_class_to_hash): Updated + declaration. + +2010-12-21 Nicola Pero <nicola.pero@meta-innovation.com> + * init.c (_objc_load_callback): Initialize with 0. (__objc_call_callback): Renamed to __objc_call_load_callback. Check _objc_load_callback only once, and if it is not set, return diff --git a/libobjc/class.c b/libobjc/class.c index cf8e4c42728..b9e8730beb2 100644 --- a/libobjc/class.c +++ b/libobjc/class.c @@ -446,11 +446,12 @@ __objc_init_class_tables (void) } /* This function adds a class to the class hash table, and assigns the - class a number, unless it's already known. */ -void + class a number, unless it's already known. Return 'YES' if the + class was added. Return 'NO' if the class was already known. */ +BOOL __objc_add_class_to_hash (Class class) { - Class h_class; + Class existing_class; objc_mutex_lock (__objc_runtime_mutex); @@ -461,21 +462,28 @@ __objc_add_class_to_hash (Class class) assert (CLS_ISCLASS (class)); /* Check to see if the class is already in the hash table. */ - h_class = class_table_get_safe (class->name); - if (! h_class) + existing_class = class_table_get_safe (class->name); + + if (existing_class) + { + objc_mutex_unlock (__objc_runtime_mutex); + return NO; + } + else { - /* The class isn't in the hash table. Add the class and assign a class - number. */ + /* The class isn't in the hash table. Add the class and assign + a class number. */ static unsigned int class_number = 1; - + CLS_SETNUMBER (class, class_number); CLS_SETNUMBER (class->class_pointer, class_number); ++class_number; class_table_insert (class->name, class); - } - objc_mutex_unlock (__objc_runtime_mutex); + objc_mutex_unlock (__objc_runtime_mutex); + return YES; + } } Class diff --git a/libobjc/init.c b/libobjc/init.c index 8e60bed97c4..0401fa432e9 100644 --- a/libobjc/init.c +++ b/libobjc/init.c @@ -879,22 +879,26 @@ void __objc_init_class (Class class) { /* Store the class in the class table and assign class numbers. */ - __objc_add_class_to_hash (class); - - /* Register all of the selectors in the class and meta class. */ - __objc_register_selectors_from_class (class); - __objc_register_selectors_from_class ((Class) class->class_pointer); - - /* Install the fake dispatch tables. */ - __objc_install_premature_dtable (class); - __objc_install_premature_dtable (class->class_pointer); - - /* Register the instance methods as class methods, this is only done - for root classes. */ - __objc_register_instance_methods_to_class (class); - - if (class->protocols) - __objc_init_protocols (class->protocols); + if (__objc_add_class_to_hash (class)) + { + /* Register all of the selectors in the class and meta class. */ + __objc_register_selectors_from_class (class); + __objc_register_selectors_from_class ((Class) class->class_pointer); + + /* Install the fake dispatch tables. */ + __objc_install_premature_dtable (class); + __objc_install_premature_dtable (class->class_pointer); + + /* Register the instance methods as class methods, this is only + done for root classes. */ + __objc_register_instance_methods_to_class (class); + + if (class->protocols) + __objc_init_protocols (class->protocols); + } + else + _objc_abort ("Module contains duplicate class '%s'\n", + class->name); } /* __objc_init_protocol must be called with __objc_runtime_mutex diff --git a/libobjc/objc-private/runtime.h b/libobjc/objc-private/runtime.h index 5fd07448183..2c34add9064 100644 --- a/libobjc/objc-private/runtime.h +++ b/libobjc/objc-private/runtime.h @@ -52,17 +52,17 @@ objc/runtime.h. */ extern "C" { #endif /* __cplusplus */ -extern void __objc_add_class_to_hash(Class); /* (objc-class.c) */ -extern void __objc_init_class_tables(void); /* (objc-class.c) */ -extern void __objc_init_dispatch_tables(void); /* (objc-dispatch.c) */ -extern void __objc_install_premature_dtable(Class); /* (objc-dispatch.c) */ -extern void __objc_resolve_class_links(void); /* (objc-class.c) */ +extern BOOL __objc_add_class_to_hash (Class); /* (objc-class.c) */ +extern void __objc_init_class_tables (void); /* (objc-class.c) */ +extern void __objc_init_dispatch_tables (void); /* (objc-dispatch.c) */ +extern void __objc_install_premature_dtable (Class); /* (objc-dispatch.c) */ +extern void __objc_resolve_class_links (void); /* (objc-class.c) */ extern void __objc_update_dispatch_table_for_class (Class);/* (objc-msg.c) */ -extern int __objc_init_thread_system(void); /* thread.c */ -extern int __objc_fini_thread_system(void); /* thread.c */ +extern int __objc_init_thread_system (void); /* thread.c */ +extern int __objc_fini_thread_system (void); /* thread.c */ extern void __objc_init_class (Class class); /* init.c */ -extern void class_add_method_list(Class, struct objc_method_list *); +extern void class_add_method_list (Class, struct objc_method_list *); /* Registering instance methods as class methods for root classes */ extern void __objc_register_instance_methods_to_class(Class); |