Re: [Qemu-devel] [PATCH] Fix conversion between 12 hours and 24 hours modes.

[Antoine Mathys]

First, the ds1338 code was in a poor state and never handled the 12 hour 
clock correctly. My first patch failed to fully fix the problem so I had 
to write a second one, but at no point did Peter or I introduce a 
regression, quite the opposite.

Second, I don't know where you got the idea that I refuse to write test 
cases. I just didn't have one ready or in the works at the time.

Third, bug 1090558 in mc146818rtc is a good example of a bug which was 
not due to insufficient testing, but to poorly structured code.

There is no point worrying about unit testing if you accept code of such 
low quality. This goes for the tests too. For instance 
cmos_get_date_time() in tests/rtc-test.c doesn't work correctly in 12 
hour mode.

Fourth, I am not interested in the PC architecture, I only wrote a fix 
for bug 1090558 because Paolo asked me to. It is nice to see that fixing 
your crappy code makes me "not a nice guy" who is making things worse. 
But don't worry, I'll focus on ARM from now on.