title: Returning IEnumerable with "ToListAsync()" inside the method
body: <p>Currently I am working with some one else's code. And this is not the first time when I see something like this: </p>
  
  <pre><code>public async MyMethod Task&lt;IEnumerable&lt;MyResult&gt;&gt; FindResults()
  {
  // some code here
  var ret = await repository.BusinessObjects.ToListAsync()
  
  return ret;
  
  }
  </code></pre>
  
  <p>my <strong>question</strong> is: Isn't it counterproductive to use <strong>IEnumerable</strong> in MyMethod return type because the result has been enumerated already and it's better to keep it this way? Wouldn't it be better to use <strong>List</strong> ? </p>
  
  <p>(I guess it also applies to non-async operations, but I decided to show this the way I see it in the code I working with just in case)</p>