GeezerVison

cc @zooba, @encukou, and @serhiy-storchaka in case you are interested in this

My only comment is the same as for the recently added pop functions: we should not use these functions as precedent for their API design (specifically how the return value is handled). When we come to decide which approach we want to standardise for the C API in general, we aren't going to be forced into choosing this design just because it was used during the period between deciding-to-decide and actually deciding.

But since there's a chance that the other recently added API could also become "warts", best to align with those since they're all on the same type. There's nothing gained by deliberate inconsistency when the functions show up right next to each other.

we should not use these functions as precedent for their API design

IMO, if the C-API WG is formed and decides soon enough, this API and all its uses in CPython might be adjusted. That might affect in-flight PRs.

PyDict_SetDefaultRef() is not principally different from PyDict_SetDefault() combined with Py_XNewRef()

With nogil, the latter is incorrect. The reference can become invalid (and the object deallocated) between the PyDict_SetDefault() and Py_XNewRef() calls.
(That doesn't apply if you know no other threads can mutate the dict, but future changes can easily invalidate assumptions like that.)

As @encukou wrote, returning a strong reference is important in nogil builds and can't be safely simulated with a subsequent Py_INCREF().

I'll put up a subsequent PR that replaces internal PyDict_SetDefault calls with PyDict_SetDefaultRef.

@serhiy-storchaka - here is an example PR that changes usages: #112211. When this PR lands, I'll rebase that PR on top of it.

Are there any outstanding concerns with this change @encukou @serhiy-storchaka?

I've added the "What's New" entry

@serhiy-storchaka - would you please merge this?

Provided for consideration of the C API WG: capi-workgroup/decisions#4.

Looks like the C API WG approved the API. @serhiy-storchaka, can we merge this PR now?

One more issue needs to be resolved. Wait another week. 😉

The issue I was talking about was the ongoing vote to promote @colesbury as a core developer. I think it would be nice if Sam merge his PR himself. Welcome on the board!