Coding path sync test: clarify comment
Change-Id: Ib40f6be7d86704d5a5dd4f5bc7dd7331f18894df
diff --git a/test/coding_path_sync.cc b/test/coding_path_sync.cc
index 30d5dd5..42c03a1 100644
--- a/test/coding_path_sync.cc
+++ b/test/coding_path_sync.cc
@@ -149,17 +149,16 @@
CompressedSource enc(i);
const aom_codec_cx_pkt_t *frame = enc.ReadFrame();
- // disable the comparison for now.
- // Re-enable it locally to help diagnosing LBD/HBD mismatches,
- // and re-enable for everybody when both coding paths match,
+
+ std::vector<int16_t> lbd_yuv = dec_lbd.decode(frame);
+ std::vector<int16_t> hbd_yuv = dec_hbd.decode(frame);
+
+ // TODO(aomedia:39): re-enable the comparison when both coding paths match,
// so they don't diverge anymore.
+ // Until then, keep doing the decoding to prevent crashes from creeping in,
+ // making this test harder to re-enable.
if (0) {
- ASSERT_EQ(dec_lbd.decode(frame), dec_hbd.decode(frame));
- } else {
- // until the comparison is re-enabled, keep doing the decoding,
- // as it still detects decoder crashes (Issue 677)
- dec_lbd.decode(frame);
- dec_hbd.decode(frame);
+ ASSERT_EQ(lbd_yuv, hbd_yuv);
}
}
}
