mikros/rust
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Rollup merge of #108917 - Ayush1325:tidy-improve, r=workingjubilee

Browse Source 
Consider target_family as pal

Currently, tidy does not consider code in target_family as platform-specific. This is erroneous and should be fixed.

r? `@workingjubilee`
This commit is contained in:
Matthias Krüger 2023-03-09 12:11:54 +01:00 committed by GitHub
commit 972ea7fd84
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/tools/tidy/src/pal.rs
View File

@ -62,6 +62,8 @@
"library/std/src/panic.rs", // fuchsia-specific panic backtrace handling "library/std/src/panic.rs", // fuchsia-specific panic backtrace handling
"library/std/src/personality.rs", "library/std/src/personality.rs",
"library/std/src/personality/", "library/std/src/personality/",
"library/std/src/thread/mod.rs",
"library/std/src/thread/local.rs",
]; ];
pub fn check(path: &Path, bad: &mut bool) { pub fn check(path: &Path, bad: &mut bool) {
@ -128,6 +130,7 @@ fn check_cfgs(
|| cfg.contains("target_env") || cfg.contains("target_env")
|| cfg.contains("target_abi") || cfg.contains("target_abi")
|| cfg.contains("target_vendor") || cfg.contains("target_vendor")
|| cfg.contains("target_family")
|| cfg.contains("unix") || cfg.contains("unix")
|| cfg.contains("windows"); || cfg.contains("windows");