[flac-dev] PATCH: Add test for metaflac --add-replay-gain

Erik de Castro Lopo mle+la at mega-nerd.com
Sun Mar 18 13:36:42 PDT 2012


Earl Chew wrote:

> I cannot reproduce the problem you see with either dash(1) or bash(1).

Just found something really werid. I opened the patch in a text editor
that doesn't support unicode and found this:

    +    awk -- '
    +    BEGIN {
    +            samplerate = '$1';

Everywhere there was supposed to be a space (hex 0a) there was a pair
(hex c2 0a). In my regular unicode aware editor, the hex c2 0a pair
just showed up as a space. I just checked and your original patch also
has these extra bytes. This may be something added by YahooMail.

With the cruft removed, the patch applied and the tests ran correctly.

I'll commit and push after a little testing.

Thanks.

Cheers,
Erik
-- 
----------------------------------------------------------------------
Erik de Castro Lopo
http://www.mega-nerd.com/


More information about the flac-dev mailing list