Namazu-devel-ja(旧)


[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: mp3 filterのメディアタイプ



臼田です。

mp3 フィルタはまだまだ改善の余地がありそうです。
ご意見ありがとうございます。

"koi_san" wrote:
> > ・処理がむだである
> > ・MP3::Infoが誤動作する
> > のどちらですか?
> 
> 数個のファイルを、MP3::Info 1.02で確認した限りでは、ID3タグの有無に関
> 係なくaudio/mpegを処理できます。
> mp3.plがget_mp3info()で取得できる情報を使っていないので、メディアタイ
> プを変更してほしいと書いたまでです。
> --------------------

メディアタイプを変えなくても問題がないという理解でよければ

情報をもっていないものに対して同じことをするのは確かに
無駄ですので
下記のようにしたいと思います。

Index: filter/mp3.pl
===================================================================
RCS file: /storage/cvsroot/namazu/filter/mp3.pl,v
retrieving revision 1.1.2.5
diff -u -r1.1.2.5 mp3.pl
--- filter/mp3.pl       27 Mar 2004 09:45:01 -0000      1.1.2.5
+++ filter/mp3.pl       29 Mar 2004 22:41:04 -0000
@@ -68,6 +68,13 @@
       = @_;
     my $cfile = defined $orig_cfile ? $$orig_cfile : '';
 
+    my $header = substr($$contref, 0, 3);
+    unless ($header =~ /ID3/) {
+       util::vprint("Couldn't find ID3 tag\n");
+        $$contref="";
+       return undef;
+    }
+
     util::vprint("Processing mp3 file ... (using MP3::Info module)\n");
 
     my $mp3;

臼田幸生