Поправил подсветку переменных и комплит в сложных случаях (так теперь он должен выдавать даже список членов для параметров лямбд типы которых не заданы явно.
Просьба потестировать данные фичи.
Так же если у кого имеется интерес, может взяться за реализацию рефакторинга "Переименование переменных". Благо теперь вроде бы для этой фичи все должно быть готово.
Есть логика намерений и логика обстоятельств, последняя всегда сильнее.
Здравствуйте, VladD2, Вы писали:
VD>Поправил подсветку переменных и комплит в сложных случаях (так теперь он должен выдавать даже список членов для параметров лямбд типы которых не заданы явно.
VD>Просьба потестировать данные фичи.
VD>Так же если у кого имеется интерес, может взяться за реализацию рефакторинга "Переименование переменных". Благо теперь вроде бы для этой фичи все должно быть готово.
Подсветка и комплишен вроде работают
А вот для рефаторинга еще не время — поиск не идеален. Например, файл Engine.Completion.n, line 204. Встаем на имя функции (completeTypeEnforcement), "Go to reference" и тишина....
Здравствуйте, Сергей Туленцев, Вы писали:
СТ>Подсветка и комплишен вроде работают СТ>А вот для рефаторинга еще не время — поиск не идеален. Например, файл Engine.Completion.n, line 204. Встаем на имя функции (completeTypeEnforcement), "Go to reference" и тишина....
Подсветка и "Go to reference" реализованы через одни и те же функции, так что вести они себя должны одинаково.
Что касается "тишины". Ну, что же поделаешь? В некоторых случаях будет тишина. В любом случае даже минимальная поддержка рефакторинга переименования принесет пользу. Даже если он будет доступен внутри методов, то уже много сил можно будет сэкономить. А уж если можно будет поля переименовывать, то и подавно.
Есть логика намерений и логика обстоятельств, последняя всегда сильнее.
Здравствуйте, VladD2, Вы писали:
VD>Что касается "тишины". Ну, что же поделаешь? В некоторых случаях будет тишина. В любом случае даже минимальная поддержка рефакторинга переименования принесет пользу. Даже если он будет доступен внутри методов, то уже много сил можно будет сэкономить. А уж если можно будет поля переименовывать, то и подавно.
Я знаю, что через одни и те же. Вот и говорю, что надо бы поиск подкрутить, а то подсветка там тоже не работает.
Здравствуйте, VladD2, Вы писали:
VD>Что касается "тишины". Ну, что же поделаешь? В некоторых случаях будет тишина. В любом случае даже минимальная поддержка рефакторинга переименования принесет пользу. Даже если он будет доступен внутри методов, то уже много сил можно будет сэкономить. А уж если можно будет поля переименовывать, то и подавно.
Таки подкрутил. Поиск и хайлайтинг по локальным функциям работает. Есть одна только проблема — хвостово-рекурсивные вызовы. Они представляются в виде TExpr.SelfTailCall(parms), то есть, нет ссылки на объявление функции. Если хотим это вхождение тоже подсвечивать, надо будет добавить такое поле.
Здравствуйте, Сергей Туленцев, Вы писали:
СТ>Таки подкрутил. Поиск и хайлайтинг по локальным функциям работает. Есть одна только проблема — хвостово-рекурсивные вызовы. Они представляются в виде TExpr.SelfTailCall(parms), то есть, нет ссылки на объявление функции. Если хотим это вхождение тоже подсвечивать, надо будет добавить такое поле.
Здорово (работоспособность пока не проверял)!
Может занялся бы добавлением рефакторинга? Там теперь не так много надо сделать. Главным образом ГУИ присобачить. Так же надо разобраться как сделать так, чтобы изменения можно было одним махом отменить. А то у нас сейчас множественный комментарий мало того, что криво работает (неровно комментарии на строках расставляет), так еще и при отмене приходится каждую строчку по отдельности откатывать . Кстати, это тоже бы поправить бы...
Есть логика намерений и логика обстоятельств, последняя всегда сильнее.
Здравствуйте, VladD2, Вы писали:
VD>Здорово (работоспособность пока не проверял)!
VD>Может занялся бы добавлением рефакторинга? Там теперь не так много надо сделать. Главным образом ГУИ присобачить. Так же надо разобраться как сделать так, чтобы изменения можно было одним махом отменить. А то у нас сейчас множественный комментарий мало того, что криво работает (неровно комментарии на строках расставляет), так еще и при отмене приходится каждую строчку по отдельности откатывать . Кстати, это тоже бы поправить бы...
Хорошо, займусь. Я похожее делал для форматтера (кстати, надо бы и его доделать, но я там заткнулся на том, как похитрее определить позицию курсора, и как его, курсор, индентить).
А поведение Comment selection меня самого раздражает.
. Кстати, это тоже бы поправить бы...
