Skip to content

Commit ca66d2c

Browse files
committed
deqp: Make code clearer
1 parent 31eaba8 commit ca66d2c

File tree

1 file changed

+8
-5
lines changed

1 file changed

+8
-5
lines changed

src/deqp/driver.volt

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -259,29 +259,32 @@ public:
259259
return;
260260
}
261261

262+
// Only rerun tests if a fewer then eight of the tests fail.
263+
eightOfTests := total / 8;
264+
262265
info(" :: Rerunning failed test(s).");
263266

264267
if (settings.noRerunTests) {
265268
info("\tRerunning tests disabled by arguments or settings");
266269
}
267270

268-
if ((total / 8) > (bad - inc)) {
271+
if (eightOfTests > (bad - inc)) {
269272
mask |= 1u << Result.Fail;
270273
mask |= 1u << Result.InternalError;
271274
} else {
272-
info("\tTo many failing tests %s", bad);
275+
info("\tTo many failing tests %s, not rerunning.", bad);
273276
}
274277

275-
if ((total / 8) > inc) {
278+
if (eightOfTests > inc) {
276279
mask |= 1u << Result.Incomplete;
277280
mask |= 1u << Result.BadTerminate;
278281
mask |= 1u << Result.BadTerminatePass;
279282
} else {
280-
info("\tTo many incomplete tests %s", inc);
283+
info("\tTo many incomplete tests %s, not rerunning.", inc);
281284
}
282285

283286
if (mask == 0) {
284-
info("\tNot rerunning any tests as there are to many.");
287+
info("\tNot rerunning any tests as there are to many failing or incomplete.");
285288
return;
286289
}
287290

0 commit comments

Comments
 (0)