namazu-ml(avocado)

[Yasuhiro MAGARA <gama@xxxxxxxxxxxxxxxxxxxxxxxx>]

まがら＠富士通オアシス開発です。

mknmzでMHonArcのインデックスをスキップするためのコードですが、
filecheck()のこの部分での判定が正しくないと思います。$NoMHonArcOpt
の単項否定(!)は不要ではないでしょうか？

diff -crBb namazu-1121.orig/src/mknmz.pl namazu-1121.new/src/mknmz.pl
*** namazu-1121.orig/src/mknmz.pl	Wed Apr 22 19:51:56 1998
--- namazu-1121.new/src/mknmz.pl	Thu Apr 23 17:33:18 1998
***************
*** 1274,1280 ****
  	"may be a BINARY file! skipped."
      } elsif ($size > $FILE_SIZE_LIMIT) {
  	"is too LARGE file! skipped.";
!     } elsif (!$NoMHonArcOpt && $cfile !~ /($MHONARC_MESSAGE_FILE)$/ 
  	     && $$contents =~ /^$MHONARC_HEADER/) {
  	"is MHonArc's index file! skipped.";
      } else {
--- 1275,1281 ----
  	"may be a BINARY file! skipped."
      } elsif ($size > $FILE_SIZE_LIMIT) {
  	"is too LARGE file! skipped.";
!     } elsif ($NoMHonArcOpt && $cfile !~ /($MHONARC_MESSAGE_FILE)$/ 
  	     && $$contents =~ /^$MHONARC_HEADER/) {
  	"is MHonArc's index file! skipped.";
      } else {


場所が前後しますが、&mhonarc_filter()の必要な場面を考えると、
document_filter()のこの部分も$NoMHonArcOptの単項否定は無い
方が良いと思います。

diff -crBb namazu-1121.orig/src/mknmz.pl namazu-1121.new/src/mknmz.pl
*** namazu-1121.orig/src/mknmz.pl	Wed Apr 22 19:51:56 1998
--- namazu-1121.new/src/mknmz.pl	Thu Apr 23 17:33:18 1998
***************
*** 606,612 ****
      my ($mhonarc_opt);
  
      $mhonarc_opt = 1 if 
! 	(!$NoMHonArcOpt && $$contents =~/^$MHONARC_HEADER/);
      if ($cfile =~ /\.($HTML_SUFFIX)$/i || $cfile =~ /$DEFAULT_FILE$/) {
  	&mhonarc_filter($contents, $weighted_str, $header) 
  	    if $mhonarc_opt;
--- 606,612 ----
      my ($mhonarc_opt);
  
      $mhonarc_opt = 1 if 
! 	($NoMHonArcOpt && $$contents =~/^$MHONARC_HEADER/);
      if ($cfile =~ /\.($HTML_SUFFIX)$/i || $cfile =~ /$DEFAULT_FILE$/) {
  	&mhonarc_filter($contents, $weighted_str, $header) 
  	    if $mhonarc_opt;


-- 
眞柄 康弘 (Magara Yasuhiro)
＃ メールアドレス変わりました。昔のアドレスで出ています。