Skip to content

WIP - add senum to thrift-parser#83

Open
ugodiggi wants to merge 2 commits intocreditkarma:masterfrom
UrbanCompass:ugo/senum
Open

WIP - add senum to thrift-parser#83
ugodiggi wants to merge 2 commits intocreditkarma:masterfrom
UrbanCompass:ugo/senum

Conversation

@ugodiggi
Copy link

A senum is like an enum but does not have the int identifier - so
it is just a string when serialized.

e.g.

senum Foo {
  VAL0,
  VAL1
}

See more details on the rationale here:
https://docs.google.com/document/d/1VuKLrMyYMhEOTCaovQQii2wd0bUhp53FcqAYUeoS-QE/edit

ugodiggi added 2 commits May 16, 2024 16:38
Also, add the relative parser tests and get them to pass.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant