When the source file contains multiple APIC frames, with equal Descriptions, we will only use the first one. This is not really what we want. We want to use the first one matching this Description and PictureType.
This patch ensures that APIC frames of different PictureType will be considered unique even if the Description was not.
Updates sequencing tests accordingly. Seems I had to get a bit more verbose in the test coverage.
n10v
commented
2 years ago
When the source file contains multiple
APICframes, with equalDescriptions, we will only use the first one. This is not really what we want. We want to use the first one matching thisDescriptionandPictureType.This patch ensures that
APICframes of differentPictureTypewill be considered unique even if theDescriptionwas not.Updates sequencing tests accordingly. Seems I had to get a bit more verbose in the test coverage.