Ну, не пользовался, баги не знаю :)

Нормальные люди на одно изменение делают один PR

Лол. У меня например может быть изменение в 10 файлах в рамках одного таска

Предлагаешь 10 PR делать?

Предлагаешь 10 PR делать?

Сам сказал в рамках одной задачи. Но если ты допустим два бага за один PR фиксишь "это моветон"

А где можно прочитать про то, что это моветон?

Всё слишком сильно зависит от того, что мы считаем за задачу

А где можно прочитать про то, что это моветон?

Там же где и про глобалки

Короче, я не хочу с тобой спорить

Ты там по-моему в каком-то своем мире живешь и программируешь, со своими идеалами кода

Надоело :C

Надоело :C

А у меня как у Снуса бзики. Он на все вопросы отвечает а я не могу терпеть плохой код.

А ты плохим кодом называешь хороший код

А ты плохим кодом называешь хороший код

А ты докажи что код хороший

А у меня как у Снуса бзики. Он на все вопросы отвечает а я не могу терпеть плохой код.

Снусу можно простить его бзики – он приносит огромное количество пользы и критика у него в 90% случаев обоснована и разумна. У тебя 10% какой-то пользы и 90% нытья/критики ни о чем

Извини, но достало, правда

Всё слишком сильно зависит от того, что мы считаем за задачу

кстати, а вот pattern и в самом деле залокалить можно было бы в рамках этого PR :)

Который pattern = "=([^%s>]*)"?

угу

[HTMLParser] [ERR]

только локалить его не в if'е, наверное, а под tagloop'ом

А это нормально, что тут два пробела?

Между скобками

(меня это всегда жутко бесит :D)

хотя не, можно и в if'е // лурканул код

А это нормально, что тут два пробела?

у тебя расплющило терминал (или в чём ты там смотришь)

там один пробел (если ты про "er] [ER")

А, да, всё ок

Снусу можно простить его бзики – он приносит огромное количество пользы и критика у него в 90% случаев обоснована и разумна. У тебя 10% какой-то пользы и 90% нытья/критики ни о чем

Откуда вдруг 10 процентов взялось?

Снусу можно простить его бзики – он приносит огромное количество пользы и критика у него в 90% случаев обоснована и разумна. У тебя 10% какой-то пользы и 90% нытья/критики ни о чем

+

@Snusmumriken

Во, призывайся и читай вопрос выше

@Snusmumriken

+

Хто ты, почему я тебя не помню

:D

Хто ты, почему я тебя не помню

Я читаю только чат. И в основном из-за одного любителя муми-троллей :D

Мы все его любим :3

@mva_name лови PR

Привет!

Доброе утро!

Чего надо-то? Проверить что-то на "хорошесть кода"?

Мы тут спорили, делает ли одна проверка глобальной переменной (не дефенишн, проверка) весь код плохим и неюзабельным

Айдар считает, что да, а мы что нет

Ну как бэ это может проверять наличие либы. Я дописывал свою векторную библиотеку на проверку наличия FFI, и использования его, если есть, и НЕиспользования его если нет. Это не просто "хороший тон", а супер-хороший тон, ибо ускорить всё при возможности = благодарность пользователей. Особенно если задокументировать "Если вы используете FFI - всё будет ещё быстрее!" В сишном коде - вообще практикуется тут и там. #ifdef WIN32 *load windows api* #ifdef POSIX *load posix api* #endif

Вот объявление глобалов - это не очень хорошо. Но что мешает внести свои три копейки, и переобъявить все глобалы в либе локально, вернув модуль и протестировав его?

Не, ну типа, local limit = GLOBALLIMIT or locallimit or default

Ага.

Вот тут GLOBALLIMIT типа не очень круто

Вот такое может быть сравнительно неплохо в том случае, если есть много версий библиотеки, и некоторые пользователи хотят обновиться, но раньше использовались глобалы. Чтобы у обновляющихся ничего не посыпалось.

Но ведь твой GLOBALLIMIT – не мой GLOBALLIMIT. Мне вот тоже может быть нужно заюзать эту переменную, но не ломая модуль. Чо делать?

Конкретно сейчас я именно так в PR и пофиксил – чтобы сохранить обратную совместимость

Нормально именовать переменные изначально ))) _[LIBNAME]_GLOBAL_LIMIT Шанс что тебе понадобится именно такая переменная - мизерен. Или заставить юзеров переписывать, да.

Мизерен. Но есть))

вообще-то, там именно так и есть

Имхо, глобалспейс вообще лучше не засорять вообще ничем

local limit = limit or htmlparser_looplimit or 1000

^

Да. Но если внезапно нет такой возможности - то хотя бы свести проблемы к минимуму.

Ну, а ибо такая возможность есть, я это и сделал :3

где первый limit перед or - локальный из коробки (т.к. не обязательный аргумент функции)

Умничка! А что в очередной раз не устраивает Айдара? : ) Сам факт этой фигни? Это как раз 0.099 часть из 0.9.

Ну, он забраковал всю либу из-за этого

а второе - проверка на глобал с именем модуля в имени переменной :)

Ну раз забраковал - ПУЩАЙ ПЕРЕДЕЛАЕТ ЛИБУ КАК ЕМУ НРАВИТСЯ ))))

Aydar, переписать одну строчку в либе - уже не судьба?

:D

Там мы еще обсуждали, как парсить все зависимости из файлов для моей графо-рисовалки, типа, вариантов require'' бесконечное множество, но тут особо пока наверное нечего предложить

Напишу вариант для require''/require('') и конвертер в dot

Я проснулся час назад и ещё не завтракал. Чутка попожже.

Сьешь блинчик

Aydar, переписать одну строчку в либе - уже не судьба?

там куда больше "не судьба"

он изначально такой "дайте либу для парсинга html, а то нет ниодной". Потом выяснилось что, цитирую "одна не кроссплатформенная" (не работает под виндой), вторая (моя) - с ужасным кодом

Ага, "раз они используют такие грязные приёмы в самом начале, значит вся либа использует такие ужасы, и ей нельзя пользоваться". Да, кстати, подобное вопиющее неприменение мозгов я тоже называю мудачеством.

когда коснулись ужасности кода - выяснилось что весь ужас в том, что проверяется наличие глобала и другой глобал я ушами прохлопал

ну и ФОРМАТИРОВАНИЕ ТАБАМИ!!!!!11111

Ребят, кто-нибудь кроме меня думает, что Айдар ведёт себя как придурок? (ну, он просто любит прилюдные голосования, а в построении вопроса нет никаких ссылок на личности, только на поведение).

Ну, коэффициент его полезности стремится к нулю. Ну и поведение не из лучших. Но оскорблять никого не буду C:

Да ладно, поведение - можно оскорблять, это же не ЛИЧНОСТЬ )))

Ну ведет себя немножко как придурок, да. Но бывают и хуже

да мы все тут не личности, а картинки с балончиками с текстом :)

У миня нет балончиков!!