Skip to content

Conversation

@jepler
Copy link

@jepler jepler commented Aug 10, 2019

.. such as namedtuple and attrtuple objects. This is the same predicate used elsewhere in the file to check for adequate compatibility between the types.

This was discovered due to crashing time.time() on the nrf port.

Closes: #2052

.. such as namedtuple and attrtuple objects.  This is the same
predicate used elsewhere in the file to check for adequate compatibility
between the types.

This was discovered due to crashing `time.time()` on the nrf port.

Closes: adafruit#2052
@jepler
Copy link
Author

jepler commented Aug 10, 2019

@dpgeorge would you like me to roll a version of this for micropython?

@tannewt tannewt added bug cpython api modules from cpython crash labels Aug 12, 2019
@tannewt tannewt added this to the 5.x.x - Bug Fixes milestone Aug 12, 2019
Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thank you!

@tannewt tannewt merged commit eb1a158 into adafruit:master Aug 12, 2019
@dpgeorge
Copy link

would you like me to roll a version of this for micropython?

Yes, it looks like a good change, but might be better to copy how objdict.c works with mp_obj_is_dict_type(o) macro? Also need to think how to write a test for this.

@jepler jepler deleted the namedtuple-tuple_get branch November 3, 2021 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug cpython api modules from cpython crash

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nrf: crash on time.time()

3 participants