<feed xmlns='http://www.w3.org/2005/Atom'>
<title>rockbox/lib/rbcodec/metadata, branch working</title>
<subtitle>My Rockbox tree</subtitle>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/'/>
<entry>
<title>Fix warning in WMA Pro and remove a c++ comment.</title>
<updated>2014-11-28T22:04:50+00:00</updated>
<author>
<name>Michael Giacomelli</name>
<email>giac2000@hotmail.com</email>
</author>
<published>2014-11-28T21:43:56+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=d924c83066759792f8a7622ed8794a97ca98bf4b'/>
<id>d924c83066759792f8a7622ed8794a97ca98bf4b</id>
<content type='text'>
Change-Id: Id9b50c1fdeca4d87f158da717de8958330f027ef
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: Id9b50c1fdeca4d87f158da717de8958330f027ef
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix FS#13009.</title>
<updated>2014-11-28T21:30:05+00:00</updated>
<author>
<name>Michael Giacomelli</name>
<email>giac2000@hotmail.com</email>
</author>
<published>2014-11-28T21:25:23+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=aa2c55e1057c0c220a1eb37c99d2c251bfe850ab'/>
<id>aa2c55e1057c0c220a1eb37c99d2c251bfe850ab</id>
<content type='text'>
This file revealed several problems with our ASF parser:

1)  The packet count in the ASF was actually a 64 bit value,
leading to overflow in very long files.

2)  Seeking blindly trusted the bitrate listed in the ASF header
rather than computing it from the packet size and number of packets.

Fix these problems and fix a few minor issues.

Change-Id: Ie0f68734e6423e837757528ddb155f3bdcc979f3
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This file revealed several problems with our ASF parser:

1)  The packet count in the ASF was actually a 64 bit value,
leading to overflow in very long files.

2)  Seeking blindly trusted the bitrate listed in the ASF header
rather than computing it from the packet size and number of packets.

Fix these problems and fix a few minor issues.

Change-Id: Ie0f68734e6423e837757528ddb155f3bdcc979f3
</pre>
</div>
</content>
</entry>
<entry>
<title>fix yellow</title>
<updated>2014-09-22T08:55:11+00:00</updated>
<author>
<name>Marcin Bukat</name>
<email>marcin.bukat@gmail.com</email>
</author>
<published>2014-09-22T08:55:11+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=da417ab93d15d0f786f86157af3715cc8c6de771'/>
<id>da417ab93d15d0f786f86157af3715cc8c6de771</id>
<content type='text'>
Change-Id: Ie3aa9b208e3f4f17d4d02f11f69839e9b381217d
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: Ie3aa9b208e3f4f17d4d02f11f69839e9b381217d
</pre>
</div>
</content>
</entry>
<entry>
<title>metadata: Add cuesheet embedded in ape tags.</title>
<updated>2014-09-22T08:20:39+00:00</updated>
<author>
<name>nialv7</name>
<email>nialv7@gmail.com</email>
</author>
<published>2014-09-15T05:33:20+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=d392da8002adfe95c8d52ac2b85143b862591aa8'/>
<id>d392da8002adfe95c8d52ac2b85143b862591aa8</id>
<content type='text'>
Change-Id: I5d9e731c3ea786fb910afbb0a5201fc68dcab9f9
Reviewed-on: http://gerrit.rockbox.org/965
Reviewed-by: Nick Peskett &lt;rockbox@peskett.co.uk&gt;
Tested: Nick Peskett &lt;rockbox@peskett.co.uk&gt;
Reviewed-by: Marcin Bukat &lt;marcin.bukat@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I5d9e731c3ea786fb910afbb0a5201fc68dcab9f9
Reviewed-on: http://gerrit.rockbox.org/965
Reviewed-by: Nick Peskett &lt;rockbox@peskett.co.uk&gt;
Tested: Nick Peskett &lt;rockbox@peskett.co.uk&gt;
Reviewed-by: Marcin Bukat &lt;marcin.bukat@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Proposed fix for FS#12878: Zero-length embedded album art prevents mp3 playback</title>
<updated>2013-12-23T16:55:15+00:00</updated>
<author>
<name>Jack Whitham</name>
<email>jack.d.whitham@gmail.com</email>
</author>
<published>2013-07-11T20:09:58+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=ca423ed0e317645c8febb8dc604743f0fd47139a'/>
<id>ca423ed0e317645c8febb8dc604743f0fd47139a</id>
<content type='text'>
see http://www.rockbox.org/tracker/task/12878

Change-Id: Ib4233c06e18d1d193dfb9e73e745ca5d174e40b2
Reviewed-on: http://gerrit.rockbox.org/507
Reviewed-by: Michael Giacomelli &lt;giac2000@hotmail.com&gt;
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
see http://www.rockbox.org/tracker/task/12878

Change-Id: Ib4233c06e18d1d193dfb9e73e745ca5d174e40b2
Reviewed-on: http://gerrit.rockbox.org/507
Reviewed-by: Michael Giacomelli &lt;giac2000@hotmail.com&gt;
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add support for flac embeded album art.</title>
<updated>2013-12-13T11:37:20+00:00</updated>
<author>
<name>Albert Song</name>
<email>albb0920@gmail.com</email>
</author>
<published>2013-12-13T09:48:48+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=f633d5ed482848826b6b081078be0414a5aaa78f'/>
<id>f633d5ed482848826b6b081078be0414a5aaa78f</id>
<content type='text'>
Change-Id: I077768f7d80b57976f9a7278b640ef67cf4f2af2
Reviewed-on: http://gerrit.rockbox.org/694
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I077768f7d80b57976f9a7278b640ef67cf4f2af2
Reviewed-on: http://gerrit.rockbox.org/694
Reviewed-by: Thomas Martitz &lt;kugel@rockbox.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix id3v2 album art if more than one image is present.</title>
<updated>2013-06-15T19:04:13+00:00</updated>
<author>
<name>Dominik Riebeling</name>
<email>Dominik.Riebeling@gmail.com</email>
</author>
<published>2013-06-15T18:56:13+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=b6ddbc41a51ef066063378ca1101b7f105f7cc6f'/>
<id>b6ddbc41a51ef066063378ca1101b7f105f7cc6f</id>
<content type='text'>
Rockbox only uses the first album art image (APIC / PIC frame) found in id3v2
tags. When a file contains more than one image the second one is ignored but
the parsealbumart() callback overwrites the already set data. This causes the
metadata structure to contain an invalid pointer to the image data, resulting
in no image shown.

Make parsealbumart() aware of this and skip parsing when an albumart image has
already been found. Fixes FS#12870.

Change-Id: Id8164f319cd5e1ee868b581f8f4ad3ea69c17f77
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Rockbox only uses the first album art image (APIC / PIC frame) found in id3v2
tags. When a file contains more than one image the second one is ignored but
the parsealbumart() callback overwrites the already set data. This causes the
metadata structure to contain an invalid pointer to the image data, resulting
in no image shown.

Make parsealbumart() aware of this and skip parsing when an albumart image has
already been found. Fixes FS#12870.

Change-Id: Id8164f319cd5e1ee868b581f8f4ad3ea69c17f77
</pre>
</div>
</content>
</entry>
<entry>
<title>SPC Codec: Have metadata parser fill in frequency and bitrate.</title>
<updated>2013-05-22T00:01:17+00:00</updated>
<author>
<name>Michael Sevakis</name>
<email>jethead71@rockbox.org</email>
</author>
<published>2013-05-22T00:01:17+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=ed24e620290ad5290fc4178f13e5c7cf611a76e0'/>
<id>ed24e620290ad5290fc4178f13e5c7cf611a76e0</id>
<content type='text'>
Change-Id: I6c72f4d1c79b1a99a11fb28e7d46886c08a56a75
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Change-Id: I6c72f4d1c79b1a99a11fb28e7d46886c08a56a75
</pre>
</div>
</content>
</entry>
<entry>
<title>Properly seek to next id3v2 frame for unsynced tags.</title>
<updated>2013-04-26T20:45:04+00:00</updated>
<author>
<name>Dominik Riebeling</name>
<email>Dominik.Riebeling@gmail.com</email>
</author>
<published>2013-04-26T20:34:59+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=370ed6de7c7596b2a1f6a2f99c8070bd179b4abd'/>
<id>370ed6de7c7596b2a1f6a2f99c8070bd179b4abd</id>
<content type='text'>
When seeking to the next id3v2 frame we need to consider if the tag has the
unsync flag set. Not doing so will likely make parsing end up in the middle of
the current frame if the frame size exceeds the upper limit set during read.
The latter usually happens for album art frames.

Fixes FS#12849.

Change-Id: Ic92853eef4374508d84df347bcc66b6661d5037d
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When seeking to the next id3v2 frame we need to consider if the tag has the
unsync flag set. Not doing so will likely make parsing end up in the middle of
the current frame if the frame size exceeds the upper limit set during read.
The latter usually happens for album art frames.

Fixes FS#12849.

Change-Id: Ic92853eef4374508d84df347bcc66b6661d5037d
</pre>
</div>
</content>
</entry>
<entry>
<title>VGM Codec: Improve time and fade behavior. Tweak minor misc.</title>
<updated>2013-03-07T00:47:05+00:00</updated>
<author>
<name>Michael Sevakis</name>
<email>jethead71@rockbox.org</email>
</author>
<published>2013-03-07T00:36:24+00:00</published>
<link rel='alternate' type='text/html' href='https://www.franklinwei.com/cgit/rockbox/commit/?id=c73894213d7f1c53eac7ea3d08ce9dbe0187aa33'/>
<id>c73894213d7f1c53eac7ea3d08ce9dbe0187aa33</id>
<content type='text'>
Prevents cutoff of tracks, especially short ones:
* Extend looped tracks by fade length to fade at start of loop repeat.
* No fade occurs for non-repeating track only having an intro.
* Uses id3.tail_trim field to store fade duration.

Use libGME built-in elapsed time reporting instead of custom calculation:
* libGME already reports in milliseconds.
* Don't advance time counter when Repeat == One. It just runs the progress
  over the length limit.

Fix a comment about sample rate and set the reported bitrate to be
accurate for 44.1 kHz stereo.

Change-Id: I3ede22bda0f9a941a3fef751f4d678eb0027344c
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Prevents cutoff of tracks, especially short ones:
* Extend looped tracks by fade length to fade at start of loop repeat.
* No fade occurs for non-repeating track only having an intro.
* Uses id3.tail_trim field to store fade duration.

Use libGME built-in elapsed time reporting instead of custom calculation:
* libGME already reports in milliseconds.
* Don't advance time counter when Repeat == One. It just runs the progress
  over the length limit.

Fix a comment about sample rate and set the reported bitrate to be
accurate for 44.1 kHz stereo.

Change-Id: I3ede22bda0f9a941a3fef751f4d678eb0027344c
</pre>
</div>
</content>
</entry>
</feed>
