mirrors
/
micropython
mirror of https://github.com/micropython/micropython.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

runtime: mp_load_method_maybe(): Don't use confusing "base" term. 

"Base" should rather refer to "base type"."Base object for attribute
lookup" should rather be just "object".

Also, a case of common subexpression elimination.
This commit is contained in:
Paul Sokolovsky 2015-03-21 00:58:07 +02:00
parent 8d51c9d376
commit 07b8dc68d6
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
py/runtime.c
View File

@ -857,13 +857,13 @@ mp_obj_t mp_load_attr(mp_obj_t base, qstr attr) {
// no attribute found, returns: dest[0] == MP_OBJ_NULL, dest[1] == MP_OBJ_NULL // no attribute found, returns: dest[0] == MP_OBJ_NULL, dest[1] == MP_OBJ_NULL
// normal attribute found, returns: dest[0] == <attribute>, dest[1] == MP_OBJ_NULL // normal attribute found, returns: dest[0] == <attribute>, dest[1] == MP_OBJ_NULL
// method attribute found, returns: dest[0] == <method>, dest[1] == <self> // method attribute found, returns: dest[0] == <method>, dest[1] == <self>
void mp_load_method_maybe(mp_obj_t base, qstr attr, mp_obj_t *dest) { void mp_load_method_maybe(mp_obj_t obj, qstr attr, mp_obj_t *dest) {
// clear output to indicate no attribute/method found yet // clear output to indicate no attribute/method found yet
dest[0] = MP_OBJ_NULL; dest[0] = MP_OBJ_NULL;
dest[1] = MP_OBJ_NULL; dest[1] = MP_OBJ_NULL;
// get the type // get the type
mp_obj_type_t *type = mp_obj_get_type(base); mp_obj_type_t *type = mp_obj_get_type(obj);
// look for built-in names // look for built-in names
if (0) { if (0) {
@ -875,11 +875,11 @@ void mp_load_method_maybe(mp_obj_t base, qstr attr, mp_obj_t *dest) {
} else if (attr == MP_QSTR___next__ && type->iternext != NULL) { } else if (attr == MP_QSTR___next__ && type->iternext != NULL) {
dest[0] = (mp_obj_t)&mp_builtin_next_obj; dest[0] = (mp_obj_t)&mp_builtin_next_obj;
dest[1] = base; dest[1] = obj;
} else if (type->load_attr != NULL) { } else if (type->load_attr != NULL) {
// this type can do its own load, so call it // this type can do its own load, so call it
type->load_attr(base, attr, dest); type->load_attr(obj, attr, dest);
} else if (type->locals_dict != NULL) { } else if (type->locals_dict != NULL) {
// generic method lookup // generic method lookup
@ -896,14 +896,14 @@ void mp_load_method_maybe(mp_obj_t base, qstr attr, mp_obj_t *dest) {
} else if (MP_OBJ_IS_TYPE(elem->value, &mp_type_classmethod)) { } else if (MP_OBJ_IS_TYPE(elem->value, &mp_type_classmethod)) {
// return a bound method, with self being the type of this object // return a bound method, with self being the type of this object
dest[0] = ((mp_obj_static_class_method_t*)elem->value)->fun; dest[0] = ((mp_obj_static_class_method_t*)elem->value)->fun;
dest[1] = mp_obj_get_type(base); dest[1] = type;
} else if (MP_OBJ_IS_TYPE(elem->value, &mp_type_type)) { } else if (MP_OBJ_IS_TYPE(elem->value, &mp_type_type)) {
// Don't try to bind types // Don't try to bind types
dest[0] = elem->value; dest[0] = elem->value;
} else if (mp_obj_is_callable(elem->value)) { } else if (mp_obj_is_callable(elem->value)) {
// return a bound method, with self being this object // return a bound method, with self being this object
dest[0] = elem->value; dest[0] = elem->value;
dest[1] = base; dest[1] = obj;
} else { } else {
// class member is a value, so just return that value // class member is a value, so just return that value
dest[0] = elem->value; dest[0] = elem->value;