Uploaded image for project: 'Picard'
  1. Picard
  2. PICARD-455

Picard setting cover art height, width and depth to 0 for FLAC files --> breaks libFLAC

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Normal Normal
    • 2.1.3
    • 2.1.2
    • None
    • None

      from http://bugs.musicbrainz.org/ticket/5798

      Hi,

      I noticed that Picard sets the following attributes to value 0 when saving a picture to a FLAC file:

      • width
      • height
      • depth

      <extract from metaflac --list output>
      METADATA block #3
      type: 6 (PICTURE)
      is last: false
      length: 41103
      type: 3 (Cover (front))
      MIME type: image/jpeg
      description:
      width: 0
      height: 0
      depth: 0
      colors: 0 (unindexed)
      data length: 41061
      data:
      00000000: FF D8 FF E0 00 10 4A 46 49 46 00 01 01 00 00 01 ......JFIF......
      00000010: 00 01 00 00 FF DB 00 43 00 05 03 04 04 04 03 05 .......C........
      ...

      This is an issue as libFLAC (more specifically FLAC__metadata_get_picture()
      ) ignores the picture inserted into the FLAC file because of this.
      I am using libFLAC to (try to) extract the picture from the FLAC file. The
      workaround I'm now using is saving the cover art as a seperate .jpg and then
      using metaflac to import the picture. This yields the following metaflac
      --list output:

      METADATA block #3
      type: 6 (PICTURE)
      is last: false
      length: 41103
      type: 3 (Cover (front))
      MIME type: image/jpeg
      description:
      width: 301
      height: 300
      depth: 24
      colors: 0 (unindexed)
      data length: 41061
      data:
      00000000: FF D8 FF E0 00 10 4A 46 49 46 00 01 01 00 00 01 ......JFIF......
      00000010: 00 01 00 00 FF DB 00 43 00 05 03 04 04 04 03 05 .......C........

      With width, height and depth set correctly, libFLAC also extracts the picture correctly.

      Would it be possible to set these attributes correctly?

      Best regards,
      Kurt

            outsidecontext Philipp Wolfer
            nikki nikki
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

                Version Package
                2.1.3