Skip to content
This repository has been archived by the owner on Nov 28, 2022. It is now read-only.

Expanding our support for integer formats to a wider variety. #245

Merged
merged 1 commit into from
Aug 5, 2019

Conversation

erunion
Copy link
Member

@erunion erunion commented Aug 3, 2019

We have a current problem in the explorer with our integer format support where we're misidentifying uint64 as int32. This resolves that, as well as expands our support to the following integer formats:

  • int8
  • uint8
  • int16
  • uint16
  • uint32
  • uint64

Ideally, we wouldn't need to hardcode these kinds of things in here, but the TSC is working through some proposals on a format registry to the spec.

Screen Shot 2019-08-02 at 11 44 13 PM

@erunion erunion added type:bug Something isn't working type:enhancement A potential new feature to be added, or an improvement we could make labels Aug 3, 2019
@erunion erunion marked this pull request as ready for review August 3, 2019 06:43
@erunion erunion merged commit bf9c2ae into master Aug 5, 2019
@erunion erunion deleted the bug/uint-support branch August 5, 2019 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type:bug Something isn't working type:enhancement A potential new feature to be added, or an improvement we could make
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants