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

Fix enum import where not nested #567

Closed
wants to merge 1 commit into from

Conversation

daemonl
Copy link

@daemonl daemonl commented Jan 31, 2022

A fix for issue #552

The solution in saltosystems@267ccc1 reverts back to an older version of the code, which then will not work with nested enums.

This should ideally work with both, but ... probably not a triple nested one.

Works in current master:

message foo {
  enum bar {
  }
}

Fails in current master:

enum bar {
}

as 'Parent' is the package/file

@daemonl daemonl force-pushed the fix-enum-import branch 2 times, most recently from d1acfe2 to f95eb29 Compare January 31, 2022 00:45
Signed-off-by: Github Actions Build <ops@embed.com>
Signed-off-by: Damien Whitten <dw@chaos404.com>

More fix

Typo

Signed-off-by: Damien Whitten <dw@chaos404.com>
@ccakes
Copy link

ccakes commented Feb 21, 2022

bumping this to request a code review and merge from a maintainer

@mattklein123 mattklein123 mentioned this pull request Feb 26, 2022
@mattklein123
Copy link
Contributor

This is fixed now.

@daemonl daemonl deleted the fix-enum-import branch August 5, 2022 18:38
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.

None yet

4 participants