[flac-dev] FIxed rest of cast-align warnings
Erik de Castro Lopo
mle+la at mega-nerd.com
Tue Jul 1 03:11:11 PDT 2014
Martijn van Beurden wrote:
> These patches indeed fix the -Wcast-align warnings, but the new
> MD5 tests fail
>
> > +++ libFLAC unit test: md5
> >
> > testing FLAC__MD5Init ... OK
> > testing that FLAC__MD5Final clears the MD5Context ... OK
> > testing FLAC__MD5Accumulate (channels=1, bytes_per_sample=1) ...
> > FAILED, expected MD5 sum b2bb8775b7d5bf59c36c8637293a4602 but
> > got 419123315a22592e3170b67d0aa26916
>
> However, using the binary on actual FLAC files (with flac -t)
> doesn't give any errors of mismatching md5 sums, so maybe the
> problem is in the test itself?
Yes, the tests were broken and were only passing by replying on
undefined behaviour.
Tests fixed, and pass on amd64/Linux, powerpc64/Linux and armh/Linux.
Erik
--
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/
More information about the flac-dev
mailing list