squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

rename parse_cfg_prefix() to has_cfg_prefix()

Browse source 
The function parse_cfg_prefix() is not really parsing. It's just checking
whether the prefix is present or not. So the new function name as suggested by
@Mark-Simulacrum is better.
This commit is contained in:
Pulkit Goyal 2018-01-16 19:00:42 +05:30
parent 1e436eb236
commit 567b07c9e6
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/tools/compiletest/src/header.rs
View file

@ -44,7 +44,7 @@ impl EarlyProps {
props.ignore = props.ignore =
props.ignore || props.ignore ||
config.parse_cfg_name_directive(ln, "ignore") || config.parse_cfg_name_directive(ln, "ignore") ||
(config.parse_cfg_prefix(ln, "only") && (config.has_cfg_prefix(ln, "only") &&
!config.parse_cfg_name_directive(ln, "only")) || !config.parse_cfg_name_directive(ln, "only")) ||
ignore_gdb(config, ln) || ignore_gdb(config, ln) ||
ignore_lldb(config, ln) || ignore_lldb(config, ln) ||
@ -566,7 +566,7 @@ impl Config {
} }
} }
fn parse_cfg_prefix(&self, line: &str, prefix: &str) -> bool { fn has_cfg_prefix(&self, line: &str, prefix: &str) -> bool {
// returns whether this line contains this prefix or not. For prefix // returns whether this line contains this prefix or not. For prefix
// "ignore", returns true if line says "ignore-x86_64", "ignore-arch", // "ignore", returns true if line says "ignore-x86_64", "ignore-arch",
// "ignore-andorid" etc. // "ignore-andorid" etc.