AdguardTeam / AdguardTeam/FiltersCompiler

2 incorrect errors in the compilation process

Abierto
#149 3 comentarios 3 reacciones 1 asignado Reclamado por @maximtop Ver en GitHub
Priority: P4
Lenguaje dominante
JavaScript
Estrellas
64
Forks
11
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.