История одного бага, который тормозил шторм последние года два.
В плагине был функционал, который высчитывал все переменные в блейд-файле. использовался в 5 местах. Все было хорошо. Потом это понадобилось в шестом месте, но там также нужно было знать и место где переменная объявлена. Ну я включил это в логику, и забыл на два года. Все эти два года народ жаловался, что редактирование блейд-файлов стало как-то подтормаживать. И вот сейчас нашелся юзер, с которым мы разобрались, узнали на что тратится куча времени процессора.
Тратится оно на просчет мест обьявления! И самое обидное, что это шестое место используется крайне редко, а первые 5 - постоянно. Но там то этот расчет и не нужен был! Чуть ли не на каждое нажатие клавиш делался огромный ненужный расчет. Исправил, релиз выложил, но стыдно будет еще долго.
В плагине был функционал, который высчитывал все переменные в блейд-файле. использовался в 5 местах. Все было хорошо. Потом это понадобилось в шестом месте, но там также нужно было знать и место где переменная объявлена. Ну я включил это в логику, и забыл на два года. Все эти два года народ жаловался, что редактирование блейд-файлов стало как-то подтормаживать. И вот сейчас нашелся юзер, с которым мы разобрались, узнали на что тратится куча времени процессора.
Тратится оно на просчет мест обьявления! И самое обидное, что это шестое место используется крайне редко, а первые 5 - постоянно. Но там то этот расчет и не нужен был! Чуть ли не на каждое нажатие клавиш делался огромный ненужный расчет. Исправил, релиз выложил, но стыдно будет еще долго.
🔥53👏20👍13🤝6😁4