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

TEnum::GetEnum("B")->GetUnderlyingType() does not following typedefs #15460

Closed
pcanal opened this issue May 8, 2024 · 0 comments · Fixed by #15461
Closed

TEnum::GetEnum("B")->GetUnderlyingType() does not following typedefs #15460

pcanal opened this issue May 8, 2024 · 0 comments · Fixed by #15461
Assignees
Labels
Milestone

Comments

@pcanal
Copy link
Member

pcanal commented May 8, 2024

As first reported in #12685:

The problem of not resolving typdefs extends to the specification of the underlying integer type, e.g.

    enum A : int {};
    TEnum::GetEnum("A")->GetUnderlyingType()  // --> (EDataType) (kInt_t) : (int) 3

but

    enum B : std::int32_t {};
    TEnum::GetEnum("B")->GetUnderlyingType()  // --> (EDataType) (kNumDataTypes) : (int) 23
@pcanal pcanal added the bug label May 8, 2024
@pcanal pcanal self-assigned this May 8, 2024
pcanal added a commit to pcanal/root that referenced this issue May 8, 2024
This fixes root-project#15460.

Extend the test accordingly.
pcanal added a commit to pcanal/root that referenced this issue May 8, 2024
This fixes root-project#15460.

Extend the test accordingly.
pcanal added a commit to pcanal/root that referenced this issue May 8, 2024
This fixes root-project#15460.

Extend the test accordingly.
pcanal added a commit to pcanal/root that referenced this issue May 8, 2024
This fixes root-project#15460.

Extend the test accordingly.
dpiparo pushed a commit that referenced this issue May 9, 2024
This fixes #15460.

Extend the test accordingly.
pcanal added a commit that referenced this issue May 9, 2024
This fixes #15460.

Extend the test accordingly.
@pcanal pcanal modified the milestones: 6.34/00, 6.32/00 May 9, 2024
pcanal added a commit that referenced this issue May 9, 2024
This fixes #15460.

Extend the test accordingly.
silverweed pushed a commit to silverweed/root that referenced this issue May 14, 2024
vepadulano pushed a commit to gwmyers/root that referenced this issue May 28, 2024
PPaye pushed a commit to PPaye/root that referenced this issue Jun 3, 2024
This fixes root-project#15460.

Extend the test accordingly.
silverweed pushed a commit to silverweed/root that referenced this issue Aug 19, 2024
This fixes root-project#15460.

Extend the test accordingly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Issues
Development

Successfully merging a pull request may close this issue.

1 participant