muennich/id3ted — pull request #5 by buffalotheory (2014-03-28 14:18:07+00:00)
Title: Fix to lookup functions in frametable.cpp - very strange iterator was re...

Greetings

This is my first pull request. I just started using id3ted; it's the only command-line tool that I found that had some support for the full range of id3v2 tags. Thanks!

However, I discovered that it was not able to read back some of the tags that it wrote, specifically the TBPM (beats-per-minute tag). It could write it, but not read it back.

I traced the issue back to the iterators in frametable.cpp. It's a very strange iterator which does not iterate in sequential order, nor does it iterate through all table entries. This commit simplifies and reduces the iterators as well as fixing the bug.

I can provide a test script as well.

Thanks!

Bryant Hansen