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

Missing Getter and Setter in C# Property #660

Closed
xuhongxu96 opened this issue Mar 18, 2021 · 0 comments · Fixed by #661
Closed

Missing Getter and Setter in C# Property #660

xuhongxu96 opened this issue Mar 18, 2021 · 0 comments · Fixed by #661
Assignees
Labels
code Source code enhancement Improvements, additions (also cosmetics)

Comments

@xuhongxu96
Copy link
Contributor

Per #550, C# has been supported. But the getters and setters cannot be parsed normally now, e.g.:

Property signature not valid, falling back to variable: string Path     = string.Empty

Getters and setters are missing in the declaration signature:

https://github.com/michaeljones/breathe/blob/4ce8c7a0d79f79f0ae3e7c7fbaa9c4fbb5c84888/breathe/renderer/sphinxrenderer.py#L1839-L1855

argsstring is empty. We should consider node.settable and node.gettable to construct { get; } and { set; }

@vermeeren vermeeren added code Source code enhancement Improvements, additions (also cosmetics) labels Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code Source code enhancement Improvements, additions (also cosmetics)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants