yocto_console_view: Fix mystery performance problem

We were pushing duplicate builder values into changes list which was causing
angular to get rather upset, triggering console warnings and heavy CPU usage.
Stop doing that. Also drop the pointless return value.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
This commit is contained in:
Richard Purdie 2020-11-29 11:01:59 +00:00
parent eacbfed5b4
commit a232d2c032

View File

@ -465,7 +465,9 @@ class Console {
change.caption = rev;
}
return change.buildersById[build.builderid].builds.push(build);
if (change.buildersById[build.builderid].builds.indexOf(build) == -1) {
change.buildersById[build.builderid].builds.push(build);
}
}
makeFakeChange(revision, when_timestamp, comments) {