空白文字しか含まない行を差分から除外したい

• Relator: (Anônimo)
• Dono: (Nenhum)
• Tipo: Feature Requests
• Estado: Fechado
• Componente: (Nenhum)
• Marcos: (Nenhum)
• Prioridade: 5 - Medium
• Gravidade: 5 - Medium
• Resolução: Accepted
• Arquivo: Nenhum

Details Responder

diffutils 2.9 以降で実装された

「空白無視、かつ空行無視が設定されているときは、半角スペースとタブしか含まない行を差分から除外」

(If -b or -w is also specified, -B now considers lines to be empty if they contain only white space.)

をWinMergeのソースにとりこんでみました。

diffutils 2.9 のソースをそのまま貼りつけはできなかったので、ver2.16.8-jp-11のソースと見比べながら書き換えた結果、下記のような感じになりました。

• hunk 内に半角スペース・タブ・改行しかなければ差分扱いにしない
• hunk 内に空白でない文字がある場合は、「空白文字しか含まない行」も含めて hunk 全体を差分とする (従来の動作に同じ)

 /* /Src/diffutils/src/util.c  */
 
 int is_blank_line (char const *pch)
 {
   while ((*pch)!='\0')
     {
       if ((*pch)=='\n' || (*pch)=='\r')
         break;
       if ((*pch)!=' ' && (*pch)!='\t')
         return 0;
       pch++;
     }
   return 1;
 }
 
 void
 analyze_hunk (struct change *hunk, 
     int *first0, int *last0, 
     int *first1, int *last1, 
     int *deletes, int *inserts, const struct file_data fd[])
 {
   int l0, l1, show_from, show_to;
   int i;
   int trivial = ignore_blank_lines_flag;
   struct change *next;
 
   show_from = show_to = 0;
 
   *first0 = hunk->line0;
   *first1 = hunk->line1;
 
   next = hunk;
   do
     {
       l0 = next->line0 + next->deleted - 1;
       l1 = next->line1 + next->inserted - 1;
       show_from += next->deleted;
       show_to += next->inserted;
 
       for (i = next->line0; i <= l0 && trivial; i++)
         {
           if (!ignore_blank_lines_flag)
             {
               trivial = 0;
             }
           else if (ignore_all_space_flag | ignore_space_change_flag)
             {
               if (is_blank_line(fd[0].linbuf[i]) == 0)
                 trivial = 0;
             }
           else if (!iseolch(fd[0].linbuf[i][0]) && fd[0].linbuf[i][0] != 0)
             {
               trivial = 0;
             }
         }
       for (i = next->line1; i <= l1 && trivial; i++)
         {
           if (!ignore_blank_lines_flag)
             {
               trivial = 0;
             }
           else if (ignore_all_space_flag | ignore_space_change_flag)
             {
               if (is_blank_line(fd[1].linbuf[i]) == 0)
                 trivial = 0;
             }
           else if (!iseolch(fd[1].linbuf[i][0]) && fd[1].linbuf[i][0] != 0)
             {
               trivial = 0;
             }
         }
     }
   while ((next = next->link) != NULL);
 
   *last0 = l0;
   *last1 = l1;
 
   /* If all inserted or deleted lines are ignorable,
      tell the caller to ignore this hunk.  */
   if (trivial)
     show_from = show_to = 0;
 
   /* WinMerge editor needs to know if there were trivial changes though,
      so stash that off in the trivial field */
   if (trivial)
     hunk->trivial = 1;
   else
     hunk->trivial = 0;
 
   *deletes = show_from;
   *inserts = show_to;
 }

Ticket History (3/4 Histories)

2021-01-28 17:34 Updated by: None

• New Ticket "空白文字しか含まない行を差分から除外したい" created

2021-01-28 23:52 Updated by: sdottaka

Comentário Responder

ご連絡ありがとうございます。 月末のリリースには間に合いませんが、次々回のリリースで取り込めるか確認してみます。

2021-01-31 00:23 Updated by: sdottaka

Comentário Responder

以下のcommit で取り込ませていただきました。ありがとうございます。

https://github.com/WinMerge/winmerge/commit/fcc7597b3b9494f4d952a1b171537d5dd6427a61

なお、is_blank_line() は、 行がNUL文字で終端すると仮定していますが、 行内にNUL文字が含まれてしまっていた場合に誤認識を避けるため、行の終端をポインタで渡すように変更しています。

2021-01-31 00:24 Updated by: sdottaka

• Estado Update from Aberto to Fechado
• Resolução Update from Nenhum to Accepted