Здравствуйте, alex-t, Вы писали:

AT>1. В чем может быть смысл такой диагностики, ведь если просто убрать из свойства set, то присваивание в вызывающем коде не скомпилится.

возможно set оставили для совместимости, чтобы код компилировался, но тот код, который использует Set сейчас не будет работать

AT>2. Если свойство только для чтения (и всегда будет только для чтения), то стОит ли его заменять функцией? Есть ли какие-то "правила хорошего тона", когда лучше свойство для чтения, а когда функция.

Насколько я знаю, советуют делать метод вместо свойства, если в get идет сложное вычисление, получение данных по сети или запрос к БД. Свойство все-таки это обычно простое свойство объекта.