ItemKey: Distinguish between Bpm and IntegerBpm

[uklotzde]

Fixes #334.

[Serial-ATA]

I believe "BPM" in Vorbis Comments and APE is also expected to be an integer value. At least that's how Picard uses it.

[uklotzde]

I believe "BPM" in Vorbis Comments and APE is also expected to be an integer value. At least that's how Picard uses it.

I am only aware of ID3v2 and MP4 which impose these restriction. MP4 by atom type and ID3v2 by standard definition.

[uklotzde]

Client still need to handle case that the integer field contains an arbitrary string, e.g. a decimal number.

[Serial-ATA]

Yeah, better to play it safe here.

This is marked draft, was there anything else planned for it?

[uklotzde]

Yeah, better to play it safe here.

This is marked draft, was there anything else planned for it?

I first wanted to publish this for discussion. If you agree we could merge it.

[Serial-ATA]

Yep, these changes are good. Thanks!