Menu

#121 Adding more ID3v2 frames

open
nobody
None
2
2004-08-22
2004-08-22
No

Depends on patches:
1004972 Preserve the characters from "eating" in
DiscDBProcessLine()
1004963 Preventing from splitting of UTF-8 characters

Includes patches:
1013403 ID3v1.1 bugfix updated

Changes:
* Using 'GripInfo' and 'EncodeTrack' structs to call
ID3v2TagFile() and ID3v2TagFile().
* Compress/escape the data while reading/writing
discDB. Additional protection to avoid forbiden writing
of newlines has been added (Replacing newlines by a
slashes).
* Avoid writing of empty tags.
* Avoid appearing of untranslated "Unknown"s in the ID3
fields.
* Added improvements to some of the existing ID3v2 fields:
- Added total numbers of tracks extension to 'Track
number/Position in set' frame.
- Comment description.
- Writing extended disc and track info to separate
comment fields.
* Added some new (to Grip ;) ) frames:
- 'Original filename'.
- 'Band/Orchestra/Accompaniment'.
- 'Length'.
- 'Media type'.
- 'File type'.

TODO:
* Filling of the language comment fields (according to
ISO-639-2).
* Increasing the "ID3 comment field" size and
replacement of 'GtkEntry' by some of multiline Gtk widgets.
* Replacing all fixed size strings in
'DiscData','GripInfo' and 'EncodeTrack' by dynamically
reallocable strings. This is important since Grip can
loss data while resending discDB entries. Except that
ID3v2 allows up to 256 megabytes field lengths. Is
there any reason forcing us to limit this to 256 bytes
(2^20 times)?

Discussion

  • Vladimir Petrov

    Vladimir Petrov - 2004-08-22

    Adding more ID3v2 frames

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-22
    • priority: 5 --> 2
     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-24

    Logged In: YES
    user_id=800933

    Ooops!
    I forgot that filenames are stored in filesystem encoding
    instead of UTF-8.
    So use version 0.2 please. :)

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-24

    update (v0.2)

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-27

    update (v0.3)

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-27

    update (v0.4)

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-27

    Logged In: YES
    user_id=800933

    In 0.3 are added more frames:
    'Encoded by'
    'File owner/licensee'

    And in 0.4:
    'Software/Hardware and settings used for encoding'
    Here is added a new function to get version info calling
    encoder and ripper with '--version' argument. I am not sure
    if its callings are placed at right places.
    TODO:
    Deactivation of all important widgets during ripping and
    separately during encoding.

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-27

    Logged In: YES
    user_id=800933

    File
    'grip-3.2.0-encodersettings.patch.gz'
    is a patch adding only 'Software/Hardware and settings used
    for encoding' field implementation to the official grip-3.2.0.
    Independent of any other patches.

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-27

    'Software/Hardware and settings used for encoding' only

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-08-29

    Logged In: YES
    user_id=800933

    Version 0.5 corrects a old data reminder bug in all early
    versions.

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-09-02

    Logged In: YES
    user_id=800933

    In 0.5 one strtoken() has at wrong place, thus eating
    compressed data instead of removing newlines from escaped. :I
    0.6 corrects that.

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-09-22

    Logged In: YES
    user_id=800933

    In 1.0 more ID3v2 frames has been added.
    But it is bad way to do full ID3v2 support, because the SAME
    code is repeated on a DIFFERENT data. There should be ONE
    code manipulating for example a data struct ARRAY by loop,
    making Grip ID3v2 suport easy expandable.
    Fixed length strings, inherited from ID3v1 time, should also
    be replaced by reallocable strings. Additional trouble thing
    are the littered everywhere array lengths.

     
  • Vladimir Petrov

    Vladimir Petrov - 2004-09-22

    Almost all ID3v2 frames

     

Log in to post a comment.