diff options
author | Alberto Gonzalez <boqwxp@airmail.cc> | 2020-05-14 22:52:07 +0000 |
---|---|---|
committer | Alberto Gonzalez <boqwxp@airmail.cc> | 2020-05-15 00:55:32 +0000 |
commit | 8297afe92588c881fbf0590a64c81af08e6f20f6 (patch) | |
tree | c826fc72447648d0c72e739e12af0d6c897070b4 /passes/cmds | |
parent | 07eecff9cca555086667666c4dbdb4c6a7133c67 (diff) | |
download | yosys-8297afe92588c881fbf0590a64c81af08e6f20f6.tar.gz yosys-8297afe92588c881fbf0590a64c81af08e6f20f6.tar.bz2 yosys-8297afe92588c881fbf0590a64c81af08e6f20f6.zip |
log: Use `dict` instead of `std::vector<std::pair>` for `log_expect_{error, warning, log}` to better express the intent that each element is unique.
Diffstat (limited to 'passes/cmds')
-rw-r--r-- | passes/cmds/logger.cc | 39 |
1 files changed, 6 insertions, 33 deletions
diff --git a/passes/cmds/logger.cc b/passes/cmds/logger.cc index 50b89d92f..c9532eced 100644 --- a/passes/cmds/logger.cc +++ b/passes/cmds/logger.cc @@ -159,39 +159,12 @@ struct LoggerPass : public Pass { log_cmd_error("Expected error message occurrences must be 1 !\n"); log("Added regex '%s' for warnings to expected %s list.\n", pattern.c_str(), type.c_str()); try { - if (type=="error") { - auto it = log_expect_error.begin(); - auto ie = log_expect_error.end(); - for (; it != ie; it++) - if (it->second.pattern == pattern) { - it->second.expected_count = count; - break; - } - if (it == ie) - log_expect_error.emplace_back(YS_REGEX_COMPILE(pattern), LogExpectedItem(pattern, count)); - } - else if (type=="warning") { - auto it = log_expect_warning.begin(); - auto ie = log_expect_warning.end(); - for (; it != ie; it++) - if (it->second.pattern == pattern) { - it->second.expected_count = count; - break; - } - if (it == ie) - log_expect_warning.emplace_back(YS_REGEX_COMPILE(pattern), LogExpectedItem(pattern, count)); - } - else if (type=="log") { - auto it = log_expect_log.begin(); - auto ie = log_expect_log.end(); - for (; it != ie; it++) - if (it->second.pattern == pattern) { - it->second.expected_count = count; - break; - } - if (it == ie) - log_expect_log.emplace_back(YS_REGEX_COMPILE(pattern), LogExpectedItem(pattern, count)); - } + if (type == "error") + log_expect_error[pattern] = LogExpectedItem(YS_REGEX_COMPILE(pattern), count); + else if (type == "warning") + log_expect_warning[pattern] = LogExpectedItem(YS_REGEX_COMPILE(pattern), count); + else if (type == "log") + log_expect_log[pattern] = LogExpectedItem(YS_REGEX_COMPILE(pattern), count); else log_abort(); } catch (const YS_REGEX_NS::regex_error& e) { |