franklinblanco
/
yuzu
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

yuzutest: Add minor comments

This commit is contained in:
Zach Hilman 2019-06-10 00:31:49 -04:00
parent 511bf3435d
commit 3a26b49c2c
2 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/yuzu_tester/service/yuzutest.cpp
View File

@ -95,7 +95,7 @@ private:
IPC::ResponseBuilder rb{ctx, 2}; IPC::ResponseBuilder rb{ctx, 2};
rb.Push(RESULT_SUCCESS); rb.Push(RESULT_SUCCESS);
finish_callback(results); finish_callback(std::move(results));
} }
std::string data; std::string data;

15
src/yuzu_tester/yuzu.cpp
View File

@ -176,7 +176,10 @@ int main(int argc, char** argv) {
finished = true; finished = true;
return_value = 0; return_value = 0;
const auto len = // Find the minimum length needed to fully enclose all test names (and the header field) in
// the fmt::format column by first finding the maximum size of any test name and comparing
// that to 9, the string length of 'Test Name'
const auto needed_length_name =
std::max<u64>(std::max_element(results.begin(), results.end(), std::max<u64>(std::max_element(results.begin(), results.end(),
[](const auto& lhs, const auto& rhs) { [](const auto& lhs, const auto& rhs) {
return lhs.name.size() < rhs.name.size(); return lhs.name.size() < rhs.name.size();
@ -187,7 +190,8 @@ int main(int argc, char** argv) {
std::size_t passed = 0; std::size_t passed = 0;
std::size_t failed = 0; std::size_t failed = 0;
std::cout << fmt::format("Result [Res Code] | {:<{}} | Extra Data", "Test Name", len) std::cout << fmt::format("Result [Res Code] | {:<{}} | Extra Data", "Test Name",
needed_length_name)
<< std::endl; << std::endl;
for (const auto& res : results) { for (const auto& res : results) {
@ -196,8 +200,8 @@ int main(int argc, char** argv) {
++passed; ++passed;
else else
++failed; ++failed;
std::cout << fmt::format("{} [{:08X}] | {:<{}} | {}", main_res, res.code, res.name, len, std::cout << fmt::format("{} [{:08X}] | {:<{}} | {}", main_res, res.code, res.name,
res.data) needed_length_name, res.data)
<< std::endl; << std::endl;
} }
@ -230,9 +234,6 @@ int main(int argc, char** argv) {
case Core::System::ResultStatus::ErrorNotInitialized: case Core::System::ResultStatus::ErrorNotInitialized:
LOG_CRITICAL(Frontend, "CPUCore not initialized"); LOG_CRITICAL(Frontend, "CPUCore not initialized");
return -1; return -1;
case Core::System::ResultStatus::ErrorSystemMode:
LOG_CRITICAL(Frontend, "Failed to determine system mode!");
return -1;
case Core::System::ResultStatus::ErrorVideoCore: case Core::System::ResultStatus::ErrorVideoCore:
LOG_CRITICAL(Frontend, "Failed to initialize VideoCore!"); LOG_CRITICAL(Frontend, "Failed to initialize VideoCore!");
return -1; return -1;