Audacious

Note: The message pops up when switching to a new song. The file indicated in the dialog is the song that has just finished playing.

I've also noticed that it is specific files that cause the message. Every time these files play, the pop-up message is displayed.

Would it be useful to send you one of the files?

Yes, please, if you could attach one of the files, that would be most helpful.

mpg123 and mp3check both agree that there is junk at the end of the file (starting at byte offset 4,402,739). It looks like it's part of an ID3 tag that some crappy tag editor left there when saving the file.

If you had looked at the console output (unfortunately, it's difficult to see it in Windows; you have to redirect it to a file), you would have seen something like this:

ERROR mpg123.cc:285 [print_mpg123_error]: mpg123 error in file:///.../16.%20Night%20Sky.mp3: Failed to find valid MPEG data within limit on resync. (code 28)

In a future version, we should look into making the error message in the GUI more useful.

These files play on every other media player I have tried with no problems, errors or warnings. I probably ripped them myself a long time ago. I have no idea what codec or software was used.

I have hundreds of these files. I would prefer that no errors are indicated to the user.

That's too bad. This makes Audacious pretty much unusable for me.

I also have some of these bad files and I agree, this warning is pretty obtrusive. Why not deactivate the dialog from appearing? Isn't a log message to the console enough, if it doesn't hinder playing?

xyz dragon wrote:

I also have some of these bad files and I agree, this warning is pretty obtrusive. Why not deactivate the dialog from appearing? Isn't a log message to the console enough, if it doesn't hinder playing?

E.g. by introducing some kind of option or just commenting out lines 364-367 in audacious/src/libaudcore/playback.cc in the latest git commit, i.e. 4558f917754dad51c04

        //aud_ui_show_error (str_printf (_("Error playing %s:\n%s"),
        // (const char *) pb_info.filename, pb_info.error_s ?
        // (const char *) pb_info.error_s : _("Unknown playback error " 
        // "(check the console for detailed error information)")));