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

Consider target_family as pal

Browse Source 
Currently tidy does not consider code in target_family as
platform-specific. I think this is erroneous and should be fixed.

Signed-off-by: Ayush Singh <ayushsingh1325@gmail.com>
This commit is contained in:
Ayush Singh 2023-03-09 01:03:07 +05:30
parent 7c306f6dcd
commit d75dae2df2
No known key found for this signature in database
GPG Key ID: 67F9C783B5A1E3D5
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");