MIPT-ILab / MDSP

[2009 – 2012] MDSP: functional simulation of a Multimedia Digital Signal Processor
MIT License
3 stars 1 forks source link

Fix regular test [FAIL] MDSP testing results 03.12.2010 19:01.18 #37

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The recent testing of main trunk showed that unit tests of funcsim fail at 
disassembly tests. That is because the disassembler format was actually changed 
to more accurate. This situation is usually called "тесты 
протухли".

The required action: modify unit tests so that they correspond to the new 
format.

As this bug has priority 'Critical' the owner is allowed to commit *fixes* 
directly to main trunk without a code review.

Original issue reported on code.google.com by grigory....@gmail.com on 3 Dec 2010 at 6:07

GoogleCodeExporter commented 9 years ago
I know the reason of this problem. I'll fix it by Monday.

Original comment by plim.the...@gmail.com on 4 Dec 2010 at 6:09

GoogleCodeExporter commented 9 years ago
This bug was fixed. Only cosmetic changes were needed. Description is in r495.

Original comment by plim.the...@gmail.com on 6 Dec 2010 at 7:59

GoogleCodeExporter commented 9 years ago
These are no cosmetic changes =)

Original comment by yuri.ba...@gmail.com on 6 Dec 2010 at 8:48

GoogleCodeExporter commented 9 years ago
Boris,

I just ran Linux tests on the new revision, they pass, so the issue may be
closed indeed. But still we (the mentors) have questions how it happened
that you broke the tests. So the discussion will proceed.

Original comment by grigory....@gmail.com on 6 Dec 2010 at 8:53

GoogleCodeExporter commented 9 years ago
I have already asked Boris about it, but in the comments to r474. If you don't 
mind I want him to answer there.

Original comment by alexander.igorevich.titov@gmail.com on 6 Dec 2010 at 9:01