Исходное сообщение
"SmartAdBlock - новый легковесный блокировщик рекламы для Chr..." Отправлено НяшМяш, 10-Мрт-19 01:08
Почему я не стал ставить это, просто посмотрев исходники: 1. Форматирование табуляцией в 8 строк; одна функция может быть отформатирована вообще без отступов, следующая - с отступами в 2 символа, а остальной файл по 8 символов 2. Нет пробелов там, где нормальные люди их ставят (понравилась лепнина вида if(str.length>0){...}else{...}) 3. Изменяют прототипы встроенных классов (String.prototype.replaceAll). Считается bad practice даже в мире вебмакак. 4. Почему-то массивы, в которые по ключу добавляют значения (да, я знаю, что массив это тот же обьект, но всё же) - suf=[];suf["com.ac"]=""; это норма. Зачем нужен именно массив, если это нормально (и красиво) складывается в обьекте? Кстати, а что это вообще за список suffix.js? 5. Лежит минифицированный jquery и я не нашёл чтобы он использовался (операции с сетью и ДОМом, которые я обнаружил, делаются вручную). Однако он вписан в манифест и в нём могут заминифицировать хоть майнера. 6. В конце концов в 2019 можно код и на ES6 написать для удобства (но это уже чисто придирка) В итоге имеем, что: 1. Код оформлен так, что ревьюить даже при желании мало кому захочется (упрт отступы, имена переменных и т.п.) 2. Проект новый (14 дней), но озаботились оформлением именного юзера (видать, чтобы авторы не спалились). Пока все коммиты только от этого юзера. На доверии работают пацаны. Однако, это всего-лишь мой внутренний аудитор включил параноика и я буду очень рад ошибиться и мы в будущем получим быстрый блокировщик рекламы.