Jump to content

Opus ReplayGain tags ignored


Recommended Posts

Link to comment
Share on other sites

56 minutes ago, LuK1337 said:

After editing tags with ExFalso, Poweramp fails to parse RG tags. I attached both files along with screenshots showing the issue.

So what changes were made by ExFalso? Seems this is the program that made a change that affected Poweramp. Does any other ReplayGain capable player have a similar issue?

Link to comment
Share on other sites

Posted (edited)
48 minutes ago, MotleyG said:

So what changes were made by ExFalso? Seems this is the program that made a change that affected Poweramp. Does any other ReplayGain capable player have a similar issue?

According to ffprobe the tags are just...in different order, and I have no idea how other players deal with that.

EDIT 1: ...Oh, they are also lowercase, maybe that's why.

EDIT 2: Yeah, when I opened libpowerampcore.so in IDA, I noticed that it does `av_dict_get(a1, "R128_TRACK_GAIN", NULL, AV_DICT_MATCH_CASE | AV_DICT_IGNORE_SUFFIX);`, so I assume that if `AV_DICT_MATCH_CASE` was dropped it'd work properly.

EDIT 3: I patched libpowerampcore.so to pass just `AV_DICT_IGNORE_SUFFIX` and that solves the issue ^.^

Edited by LuK1337
Link to comment
Share on other sites

1 hour ago, LuK1337 said:

EDIT 1: ...Oh, they are also lowercase, maybe that's why.

Yes, I was just checking the files and noticed the change in case.

Good:
image.png#

Bad:
 image.png

We've seen that causing issues before, with AlbumGain in particular being case sensitive. As you say, I'm sure it can be coded around in Poweramp to make these tags case insensitive too, but for now just stick to UPPER case.

Andre

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...