AdguardTeam / AdguardTeam/FiltersCompiler

2 incorrect errors in the compilation process

オープン
#149 コメント 3 件 リアクション 3 件 担当者 1 名 @maximtop が担当を希望しています GitHub で見る
Priority: P4
主要言語
JavaScript
スター
64
フォーク
11
PR マージ指標
30日以内にマージされた PR はありません

説明

When looking into https://raw.githubusercontent.com/AdguardTeam/FiltersRegistry/master/filters/ThirdParty/filter_250_DandelionSproutAnnoyances/diff.txt, I've worked out (or at least looked into) various excluded entries, but there are 2 excluded entries that I feel should've been allowed by the compiler, but which aren't:

1) Regarding `Error: "Unknown pseudo class: p:-moz-only-whitespace" in the rule: "##p:-moz-only-whitespace"`, Mozilla has supported `:-moz-only-whitespace` for 20 years now, so I at least presume that the Firefox team didn't intend it to be treated as merely a pseudo class. I therefore feel that AdGuard should treat it as a normal class.

2) Regarding `Error: "Unknown modifier: }-\w{10" in the rule: "$removeparam=/^kk=\w{6,}-\w{10,}-\w{5,}$/"`, I can only presume there's a glitch somewhere in the compiler that makes it think that `}-\w{10` was to be looked at as if it was a modifier.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。