__missing__ not completely implemented in dictobject.c

[bvukov]

BPO	6468
Nosy	@benjaminp

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2009-07-12.17:03:00.893>
created_at = <Date 2009-07-12.16:50:48.666>
labels = ['type-bug']
title = '__missing__ not completely implemented in dictobject.c'
updated_at = <Date 2009-07-12.17:03:00.886>
user = 'https://bugs.python.org/bvukov'

bugs.python.org fields:

activity = <Date 2009-07-12.17:03:00.886>
actor = 'benjamin.peterson'
assignee = 'none'
closed = True
closed_date = <Date 2009-07-12.17:03:00.893>
closer = 'benjamin.peterson'
components = ['None']
creation = <Date 2009-07-12.16:50:48.666>
creator = 'bvukov'
dependencies = []
files = []
hgrepos = []
issue_num = 6468
keywords = []
message_count = 2.0
messages = ['90450', '90451']
nosy_count = 2.0
nosy_names = ['bvukov', 'benjamin.peterson']
pr_nums = []
priority = 'normal'
resolution = 'wont fix'
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue6468'
versions = ['Python 2.5']

[bvukov]

For some reason functions PyDict_GetItem ( and PyDict_GetItemString )
don't try to check for '__missing__' on subclass, but
dict_subscript(dictobject *mp, register PyObject *key) does.
Only in this function a failure to get a value using
ma_lookup checks afterwards using !PyDict_CheckExact for subclass
definition of '__missing__', and then returns a value.
Is this intended or just a half implementation ?

[benjaminp]

Yes, this is intended. PyDict_GetItem already bypasses user __getitem__
methods, so it's logical it would do the same for __missing__.