Namazu-devel-ja(旧)
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: mp3 filterのメディアタイプ
- From: Yukio USUDA <m6694ha392t@xxxxxxxxxxxxxxx>
- Date: Tue, 30 Mar 2004 07:48:50 +0900
- X-ml-name: namazu-devel-ja
- X-mail-count: 04001
- References: <000c01c415b5$8f1309f0$0201a8c0@misa>
臼田です。
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;
臼田幸生