Function without return type

In my project (which I inherited from someone) there are a lot of functions like:

Public Function DoSomething(ByVal s As String)
    ' Do something to public properties
End Function

And they are called like this:

DoSomething(s)

So the return value is ignored (which is object, as I see in the docs). Is it safe to change al开发者_JAVA百科l these functions to Subs? Could I break something which isn't so obvious?

---

Should be safe. That's what Subs are for, this is really bad style by your predecessor.

---

Is there any specific reason you want to convert to Subs? From your code the Function has no return type - in this situation I can't see there being any problem.