blender
/
blender
128
396
Fork 569
Code Issues 6.2k Pull Requests 755 Projects 19 Wiki Activity

Cleanup: Else after return, other simplifications 

`std::stringstream` already returns a `std::string`, and there is no
particular reason to use short here instead of int.
This commit is contained in:
Hans Goudey 2021-11-23 12:49:45 -05:00
parent c09e8a3590
commit 62b50c612f
1 changed files with 12 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
source/blender/editors/space_node/node_draw.cc
View File

@ -1664,24 +1664,22 @@ static std::string node_get_execution_time_label(const SpaceNode *snode, const b
if (exec_time_us == 0) {
return std::string("-");
}
else if (exec_time_us < 100) {
if (exec_time_us < 100) {
return std::string("< 0.1 ms");
}
else {
short precision = 0;
/* Show decimal if value is below 1ms */
if (exec_time_us < 1000) {
precision = 2;
}
else if (exec_time_us < 10000) {
precision = 1;
}
std::stringstream stream;
stream << std::fixed << std::setprecision(precision) << (exec_time_us / 1000.0f);
return std::string(stream.str() + " ms");
int precision = 0;
/* Show decimal if value is below 1ms */
if (exec_time_us < 1000) {
precision = 2;
}
return std::string("");
else if (exec_time_us < 10000) {
precision = 1;
}
std::stringstream stream;
stream << std::fixed << std::setprecision(precision) << (exec_time_us / 1000.0f);
return stream.str() + " ms";
}
struct NodeExtraInfoRow {