Skip to content

Commit 2750a7b

Browse files
committed
py/objdict: Actually provide the key that failed in KeyError exception.
The failed key is available as exc.args[0], as per CPython.
1 parent a3edeb9 commit 2750a7b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

py/objdict.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ mp_obj_t mp_obj_dict_get(mp_obj_t self_in, mp_obj_t index) {
162162
mp_obj_dict_t *self = MP_OBJ_TO_PTR(self_in);
163163
mp_map_elem_t *elem = mp_map_lookup(&self->map, index, MP_MAP_LOOKUP);
164164
if (elem == NULL) {
165-
nlr_raise(mp_obj_new_exception_msg(&mp_type_KeyError, "<value>"));
165+
nlr_raise(mp_obj_new_exception_arg1(&mp_type_KeyError, index));
166166
} else {
167167
return elem->value;
168168
}
@@ -178,7 +178,7 @@ STATIC mp_obj_t dict_subscr(mp_obj_t self_in, mp_obj_t index, mp_obj_t value) {
178178
mp_obj_dict_t *self = MP_OBJ_TO_PTR(self_in);
179179
mp_map_elem_t *elem = mp_map_lookup(&self->map, index, MP_MAP_LOOKUP);
180180
if (elem == NULL) {
181-
nlr_raise(mp_obj_new_exception_msg(&mp_type_KeyError, "<value>"));
181+
nlr_raise(mp_obj_new_exception_arg1(&mp_type_KeyError, index));
182182
} else {
183183
return elem->value;
184184
}
@@ -283,7 +283,7 @@ STATIC mp_obj_t dict_get_helper(mp_map_t *self, mp_obj_t key, mp_obj_t deflt, mp
283283
if (elem == NULL || elem->value == MP_OBJ_NULL) {
284284
if (deflt == MP_OBJ_NULL) {
285285
if (lookup_kind == MP_MAP_LOOKUP_REMOVE_IF_FOUND) {
286-
nlr_raise(mp_obj_new_exception_msg(&mp_type_KeyError, "<value>"));
286+
nlr_raise(mp_obj_new_exception_arg1(&mp_type_KeyError, key));
287287
} else {
288288
value = mp_const_none;
289289
}

0 commit comments

Comments
 (0)