Skip to content

Fix compiler and clippy warnings#105

Merged
BusyJay merged 1 commit intotikv:mainfrom
miried:fix-warning
Oct 16, 2024
Merged

Fix compiler and clippy warnings#105
BusyJay merged 1 commit intotikv:mainfrom
miried:fix-warning

Conversation

@miried
Copy link

@miried miried commented Oct 15, 2024

  • Added hints for jemallocator_docs and prefixed configs.
  • Removed superfluous checks on LE variants of mips and powerpc64 arch [1].
  • Fixed various indentations.

[1] target_arch mipsel and powerpc64le don't exist (anymore?). To check for the LE variants specifically, it would need the check on target_endian (https://doc.rust-lang.org/reference/conditional-compilation.html#target_endian) (cf. this comment: kata-containers/kata-containers#10072 (comment))

caveat: I have no hardware to test the change on the 2 affected architectures.

Signed-off-by: Michael Rieder <[email protected]>
@BusyJay BusyJay merged commit 5787ab4 into tikv:main Oct 16, 2024
@BusyJay
Copy link
Member

BusyJay commented Oct 16, 2024

Thank you!

@miried miried deleted the fix-warning branch October 21, 2024 10:01
x86pup pushed a commit to x86pup/jemallocator that referenced this pull request Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants