コメントが正しく認識されない
ご連絡ありがとうございます。 こちらでも再現できました。 まだ修正できるか見ていませんが、修正できるようでしたら次のバージョンに反映されると思います。
Reply To sdottaka
ご連絡ありがとうございます。 こちらでも再現できました。 まだ修正できるか見ていませんが、修正できるようでしたら次のバージョンに反映されると思います。
ありがとうございます。 助かります。
加えて、一部お伝えした内容に誤りがありました。
以下、訂正いたします。
2. コメントの差異を無視するオプションを有効にしていると、1行目の行末の /** 以降が、比較されない
↓
2. コメントの差異を無視するオプションを有効にしていると、以下のような状況で比較が無視される
file : a.c
file : b.c
1. 1行目の行末の /** からシンタックスがコメントにならない。
一番目の件に関しては、修正を行いプレリリース版をアップロードしております。
https://osdn.net/projects/winmerge-jp/releases/73184
2. コメントの差異を無視するオプションを有効にしていると、以下のような状況で比較が無視される
二番目の件に関しては、現在のコメント内の差異を無視するロジックで一部、行内に複数のコメントブロックが存在することを考慮できていないような箇所があり、これを修正すると直せるのではないかと考えていますが、かなりまずい不具合と思いつつ、私には複雑すぎてなかなか手が出しにくい状況です。
そもそも、今のロジックを直したとしてもまだうまくいかないケースがあるのではないかという疑念と コメントの差異を無視するロジックのブロックコメントを検出するロジックが、シンタックスハイライトのブロックコメントの検出ロジックとまったく別ものであり、 HTMLなどのコメントは検出できていないことから、シンタックスハイライトのロジックベースにソースコード完全に書き換えたいという思いがあり、 時間をいただきたいと思っています。
一番目の件に関しては、修正を行いプレリリース版をアップロードしております。
ありがとうございます。試してみます。
二番目の件に関しては、現在のコメント内の差異を無視するロジックで一部、行内に複数のコメントブロックが存在することを考慮できていないような箇所があり、これを修正すると直せるのではないかと考えていますが、かなりまずい不具合と思いつつ、私には複雑すぎてなかなか手が出しにくい状況です。
そもそも、今のロジックを直したとしてもまだうまくいかないケースがあるのではないかという疑念と コメントの差異を無視するロジックのブロックコメントを検出するロジックが、シンタックスハイライトのブロックコメントの検出ロジックとまったく別ものであり、 HTMLなどのコメントは検出できていないことから、シンタックスハイライトのロジックベースにソースコード完全に書き換えたいという思いがあり、 時間をいただきたいと思っています。
状況把握しました。
心待ちにしております。。。
Reply To sdottaka
そもそも、今のロジックを直したとしてもまだうまくいかないケースがあるのではないかという疑念と コメントの差異を無視するロジックのブロックコメントを検出するロジックが、シンタックスハイライトのブロックコメントの検出ロジックとまったく別ものであり、 HTMLなどのコメントは検出できていないことから、シンタックスハイライトのロジックベースにソースコード完全に書き換えたいという思いがあり、 時間をいただきたいと思っています。
横から失礼いたします。
HTML内 のコメントについて、script要素のすぐ内側にある <!-- --> を比較対象から外すのはまずい気がします。
「script 要素内では /* */ と// をコメント扱いとする」ようにできれば理想ですが、それをするには単体の javascript の シンタックスハイライトを先に実装する必要がありそうです。
作りたい気持ちは山々なれど、現行のソースコードの把握が追い付かず……
遅くなってすみません。コメントフィルタをシンタックスハイライトベースに書き直し、本問題に対処しました。 プレリリース版 2.16.8+-jp-6 を作成しています。
https://github.com/sdottaka/winmerge-v2-jp/releases/tag/2.16.8%2B-jp-6
HTML内 のコメントについて、script要素のすぐ内側にある <!-- --> を比較対象から外すのはまずい気がします。
>「script 要素内では /* */ と// をコメント扱いとする」ようにできれば理想ですが、それをするには単体の javascript の シンタックスハイライトを先に実装する必要がありそうです。
まだ、こちらには対応できていないため、こちらもまた時間がかかるかもしれませんが、対処していきたいと思います。
>script 要素内では /* */ と// をコメント扱いとする」ようにできれば理想ですが、それをするには単体の javascript の シンタックスハイライトを先に実装する必要がありそうです。
こちらのほうも プレリリース版2.16.8+-jp-7 で対応しました https://github.com/sdottaka/winmerge-v2-jp/releases/tag/2.16.8%2B-jp-7
言語 : C言語
Doxygenのコメントで以下の用な感じのコメントがあると、以下の挙動になる
1. 1行目の行末の /** からシンタックスがコメントにならない。
2. コメントの差異を無視するオプションを有効にしていると、1行目の行末の /** 以降が、比較されない
/***************//**
etc...
******************/