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.

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

Document enums #13

Open
melix99 opened this issue Mar 26, 2022 · 3 comments
Open

Document enums #13

melix99 opened this issue Mar 26, 2022 · 3 comments

Comments

@melix99
Copy link
Collaborator

melix99 commented Mar 26, 2022

Currently this is the missing piece for having a 100% documented library. To do that we need to parse the "@Class" tag in the .tl file first.

@x2bool
Copy link

x2bool commented May 16, 2022

Hi! Perhaps I could help?

What do you think about porting parser to use something like https://github.com/Geal/nom ?

I could rewrite parser to produce full AST including @ Class definitions.

I have experience with parsing .tl files using parser combinators: https://github.com/egramtel/tdsharp/blob/master/TdLib.CodeGen/Parser.fs (.NET / C# / F#)

@x2bool
Copy link

x2bool commented May 16, 2022

Having full AST could also help with removing 1-variant enums #14

@melix99
Copy link
Collaborator Author

melix99 commented May 16, 2022

Hi! Perhaps I could help?

What do you think about porting parser to use something like https://github.com/Geal/nom ?

I could rewrite parser to produce full AST including @ Class definitions.

I have experience with parsing .tl files using parser combinators: https://github.com/egramtel/tdsharp/blob/master/TdLib.CodeGen/Parser.fs (.NET / C# / F#)

Hi!

I think that would be great! I'd be more than happy to review/merge that.

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

No branches or pull requests

2 participants