Stabilize hex_literal_case#6935
Conversation
| let h3 = -0xCAFE_5EA7; | ||
| let h4 = -0xCAFE_F00Di32; |
There was a problem hiding this comment.
Thanks for adding these negative value tests. Int literals can also be written with a _{type} suffixes like _u32. Just to cover all the bases I think we should add some test cases with those kinds of suffixes.
There was a problem hiding this comment.
Thanks for trying to get this option stabilized. I think hex_literal_case is a good candidate for stabilization. I don't see any bugs in the backlog, and I assume that it's being used by projects out in the wild, but we should try to confirm if there's a good amount of real world usage.
Also, For consistency with other config options, Let's move all of these tests to tests/{source|target}/configs/hex_literal_case
@rustbot author
|
Reminder, once the PR becomes ready for a review, use |
Stabilizing
hex_literal_caseThis PR try to address the issue #5081