Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add more information about tagged types in the Encoding pretty printer #72

Open
axman6 opened this issue Jun 1, 2016 · 2 comments
Open

Add more information about tagged types in the Encoding pretty printer #72

axman6 opened this issue Jun 1, 2016 · 2 comments
Labels
A-enhance E-medium P-medium

Comments

@axman6
Copy link
Contributor

axman6 commented Jun 1, 2016

As I noted in #69, it would be nice to add some more info to the pretty printer about the content of tags, so we can add notes such as # tag(0)[time string], # tag(64)[uint8 Typed Array] (see #62 and #63). I'm happy to do this one day when we support more of the extensions.
@dcoutts
Copy link
Member

dcoutts commented Jun 1, 2016

Right, sounds sensible. I think we'd just need a built-in mapping of known tags.

@bgamari
Copy link
Collaborator

bgamari commented Aug 1, 2017

Patches welcome! 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-enhance E-medium P-medium
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@thoughtpolice @axman6 @bgamari @dcoutts