Ответ 1
Должно быть безопасно. Для того, для чего предназначены Subs, это действительно плохой стиль со стороны вашего предшественника.
В моем проекте (который я унаследовал от кого-то) есть много таких функций, как:
Public Function DoSomething(ByVal s As String)
' Do something to public properties
End Function
И они называются так:
DoSomething(s)
Поэтому возвращаемое значение игнорируется (это объект, как я вижу в документах). Можно ли изменить все эти функции на Subs? Могу ли я сломать что-то, что не так очевидно?
Должно быть безопасно. Для того, для чего предназначены Subs, это действительно плохой стиль со стороны вашего предшественника.
Есть ли какая-то конкретная причина, по которой вы хотите конвертировать в Subs? Из вашего кода функция имеет тип возврата - в этой ситуации я не вижу проблемы с ней.