AudioGenie Forum
September 08, 2010, 04:49:32 *
Willkommen Gast. Bitte einloggen oder registrieren.
Haben Sie Ihre Aktivierungs E-Mail übersehen?

Einloggen mit Benutzername, Passwort und Sitzungslänge
News: Neue Version 2.0.1.3 der AudioGenie DLL verfügbar!
 
Seiten: [1]
  Drucken  
Autor Thema: First test of Delphi Wrapper (MP4)  (Gelesen 242 mal)
hamsen
Newbie
*
Beiträge: 42



Profil anzeigen
« am: Februar 19, 2010, 06:20:56 »

I have tested reading MP4 files using Delphi.
These are my comments:

1) AUDIOAnalyzeFileW returns a boolean, should return smallInt.
2) MP4_CDSETNUMBER looks for ' dsk', should look for 'disk'.
3) Picturehandling should include a posibility to check for picturetype or change the fileextension of the passed filename. Otherwise special handling afterwards is necessary.
4) The atom 'cpil' can also contain the value 'true'.
5) No userdefined tagnames. The MP4 atom system has a number of predefined tags (all 4 bytes). But you are free to make them as long as you wish. Therefore it would be simpler and more flexible to do it like this:

a) MP4TextFrameW['disk']
b) MP4TextFrameW['©alb']
c) MP4TextFrameW['replaygain_track_gain']
etc.

6) Reading the picture works fine.

Regards
Thomas

Gespeichert
audioxp
Newbie
*
Beiträge: 39


Profil anzeigen WWW
« Antworten #1 am: Februar 20, 2010, 04:02:30 »

Concerning "AudioAnalyzeFileW": I've changed the datatype (in the wrapper) from smallint to boolean, because it is more "native" to work with boolean than this "-1" stuff. This is also the case for some more functions that return simply "-1=true, 0=false.
Thought this would make it easier; if you don't like it, feel free to change the wrapper :-)

The other issues belong to Stefan, I think...

Best regards
Ulrich (I've coded the current version of the wrapper)
Gespeichert
hamsen
Newbie
*
Beiträge: 42



Profil anzeigen
« Antworten #2 am: Februar 20, 2010, 05:48:00 »

Concerning "AudioAnalyzeFileW": I've changed the datatype (in the wrapper) from smallint to boolean, because it is more "native" to work with boolean than this "-1" stuff. This is also the case for some more functions that return simply "-1=true, 0=false.
Thought this would make it easier; if you don't like it, feel free to change the wrapper :-)

The other issues belong to Stefan, I think...

Best regards
Ulrich (I've coded the current version of the wrapper)

I think you forget, that the return value gives the type of Audiofile.
Gespeichert
audioxp
Newbie
*
Beiträge: 39


Profil anzeigen WWW
« Antworten #3 am: Februar 20, 2010, 06:30:08 »

You are right, sorry, but with all these functions I could not check every result type :-(

I'll send Stefan a new version of the wrapper...
Gespeichert
Stefan
Administrator
Sr. Member
*****
Beiträge: 425



Profil anzeigen WWW
« Antworten #4 am: Februar 20, 2010, 07:08:57 »

Hi,

that's speed.. it is online now Zwinkernd
Gespeichert

Seiten: [1]
  Drucken  
 
Gehe zu:  

UseBB Port by Gaia Modified & Upgraded by Croco Powered by SMF 1.1.11 | SMF © 2006, Simple Machines LLC