I have been doing a huge conversion project on a VB.Net app to C#. Yes I know, there isn't much functional difference, but we are standardizing this project before we go further.

While porting, I have been finding things like this:
Code:
        If e.KeyCode = 46 Then
            'Do something...
        End If
That was obviously from a old VB6 programmer bringing their old habbits to .Net. Look at the readability of it. Instead do this for readablity and (possibly in the future) portability.
Code:
        If e.KeyCode = Keys.Delete Then
            'Do something...
        End If
I find things like this everywhere. Another example is 'vbCrLf'. Use the Environment.Newline in your code in place of that. It helps readability, and again, if they ever ported the framework, you would know that your code is calling a .Net framework class, which should be ported correctly instead of some obscure thing that may not be related on another system. I know this is a big if, but just look at the readablity of it, that alone is the biggest reason.

Sorry for the rant, I am just pretty frustrated at the moment, and thought I would share what the other side of the fence feels sometimes.

Just to note, I am a VB.Net/C# developer, so I have no biases against VB developers, I just don't like the habbits that are carried over from VB6.