Calling methods on uninitialized objects crashes MicroPython

[laurensvalk]

On closer inspection, [the crash] isn't due to inheritance per se. It also crashes with simply PUPDevice.read(PUPDevice, 0). Simpler types (StopWatch) also misbehave to a lesser extent.
It's perhaps just because we don't assert the type of self in each of our methods.

Originally posted by @laurensvalk in #777 (comment)

[laurensvalk]

I see your WIP commmit now. That's nice for a more descriptive error, but we shouldn't need it to just fix the crash. Just enabling MICROPY_BUILTIN_METHOD_CHECK_SELF_ARG suffices to raise a TypeError.

Originally posted by @laurensvalk in #777 (comment)

[laurensvalk]

It only fixes crashing in cases like to list.append(str, 0) and only for MicroPython builtins, not in any Pybricks code. The WIP branch extends it to fix Pybricks code like PUPDevice.read(PUPDevice, 0).

Originally posted by @dlech in #777 (comment)

[laurensvalk]

On master:

PUPDevice.read(PUPDevice, 0)

Crashes.

Enable MICROPY_BUILTIN_METHOD_CHECK_SELF_ARG

PUPDevice.read(PUPDevice, 0)
>>> TypeError: argument should be a '' not a 'type'

WIP Branch

PUPDevice.read(PUPDevice, 0)
>>> TypeError: argument should be a '' not a 'type'

This also sets MICROPY_BUILTIN_METHOD_CHECK_SELF_ARG but that doesn't do anything extra because mp_check_self expands to either nothing or an assert.

[laurensvalk]

It looks like MICROPY_BUILTIN_METHOD_CHECK_SELF_ARG enables the standard check but disables mp_check_self; they should not both be used in this form.