submatch対応 #8 #33
submatch対応 #8 #33
Conversation
|
（seqenceじゃない場合に）確認ミスでまだできてませんでした。。。。 |
|
|
|
確かにそうですね。。。ありがとうございます！ |
|
差分を見るなら https://github.com/n4o847/seccamp-redos/compare/submatch とかですかね プルリクがこんなんなってることの解消法は、わからず…… |
|
ぽちぽちいじってたらいい感じになりました |
|
masterとの差分がうまく表示されないことを防ぐために, 自分の編集ブランチをpushする前に, masterブランチで 根本君の件で気になって調べたところ, プルリクのFile Changedがマージベースとの差分を表示しているらしくて, それでマージベースを調べたら, 最新のmasterのコミット番号 |
|
なるほど、ありがたいです。 |
|
偽判定残り
|
|
これまでに行っていた方法は間違っていたので大幅に変更し、 |
src/enfa.ts
Outdated
| if ( | ||
| this.pattern.child.type === 'Capture' || | ||
| this.pattern.child.type === 'Group' | ||
| ) { |
まあそんな正規表現あんまり無いだろうということで放っといてもいい気もします |
|
細かい所まで見ていただきありがたいです。 |
|
sequenceの処理順を少し変えたら、 |
src/enfa.ts
Outdated
| ) { | ||
| // 遷移を削除 | ||
| this.transitions = this.transitions.filter( | ||
| ([s, n, d]) => s !== source || n !== node || d !== dest, |
There was a problem hiding this comment.
| ([s, n, d]) => s !== source || n !== node || d !== dest, | |
| ([s, n, d]) => !(s === source && n === node && d === dest), |
もしよければここも変えてほしいです
src/enfa.ts
Outdated
| ) { | ||
| // 元の遷移を削除 | ||
| this.transitions = this.transitions.filter( | ||
| ([s, n, d]) => s !== source || n !== node || d !== dest, |
There was a problem hiding this comment.
| ([s, n, d]) => s !== source || n !== node || d !== dest, | |
| ([s, n, d]) => !(s === source && n === node && d === dest), |
これ初期状態からpatChildへの遷移を消すという意味だと思うのですが意図が伝わりやすいようにこっちの方が良さそうです(完全に僕の好みですが)
There was a problem hiding this comment.
確かに、そのほうが可読性が高いと思いました。ありがとうございます
No description provided.