title: Function without return type
body: <p>In my project (which I inherited from someone) there are a lot of functions like:</p>
  
  <pre><code>Public Function DoSomething(ByVal s As String)
      ' Do something to public properties
  End Function
  </code></pre>
  
  <p>And they are called like this:</p>
  
  <pre><code>DoSomething(s)
  </code></pre>
  
  <p>So the return value is ignored (which is object, as I see in the docs). Is it safe to change all these functions to Subs? Could I break something which isn't so obvious?</p>