-
Notifications
You must be signed in to change notification settings - Fork 278
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
avoid LTO issues #2681
avoid LTO issues #2681
Conversation
Fixes: #2680 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The difference between the two structs is that one has a uint32_t
field and the other long
. I'd like to remove all occurrences of long
anyway, so I suggest we use uint32_t
in both. With that change, this struct could potential be moved to a shared header file.
Done. |
Place structs inside the places where used. Signed-off-by: Rosen Penev <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG2M
@Mergifyio backport 0.28.x |
✅ Backports have been created
|
Place structs inside the places where used.