Pull to refresh

Comments 12

Вот такие предупреждения смущают:
my $b = shift; #It's a very bad practice to declare built-in variable as our/my/state
my ($r, $tpl) = @_; #Current variable declaration shadows previous declaration of the same variable
1. Чем смущает? Подобные конструкции это лишний шанс выстрелить себе в ногу. Плагин сделан для работы над большими проектами, для мелких скриптов он просто не нужен. В большом проекте подобное недопустимо.
2. Все что смущает можно отключить.
Разве переменная $b — built-in?
Не знаю насколько большой/маленький у нас проект (но далеко не «маленький скрипт»), но в чем подвох этой конструкции — «my ($r, $tpl) = @_;»?
$a и $b — глобальные переменные. используются, например, для сортировки.
Подвох не в конструкции. А в том что вы уже определили переменную $r или $tpl в данном скоупе.
Насчет $r и $tpl — да, забавный момент. Никогда бы не подумал, что человек может на автомате my вставлять, не задумываясь о видимости переменной.
А вот насчет $a и $b — да, они используются при сортировке, но что-то я не встречал описание ограничений на их использование.
Их нет. Но присваивать что-либо $_ $a, $b — опасно. Этот инспекшн сделан для того чтобы избежать подобных вещей. Опять же — все отключаемо.
В чем причина не обновления плагина для IDEA 14?
Причины две:
1. Это мое свободное время за которое никто не платит и две версии поддерживать — времени нужно больше, а и так не хватает на реализацию всех идей.
2. Статистически скачек для 14 версии очень мало в сравнении с 15.
Скромно: я как раз вам перевел в поддержку через paypal.
Спасибо.
Огромное спасибо за поддержку!
Sign up to leave a comment.

Articles