[Style] Target-typed new in the codebase

[Youssef1313]

We have excessive use of target-typed new, which (IMO) makes the code much harder to read in many cases where the type isn't apparent.

I would consider switching the code style to even disallow its usage completely. It's a personal opinion though and there is no right/wrong answer here.

[nohwnd]

Sometimes I find it hard, sometimes I don't. thinking about it, it is maybe harder than it was with var before.

Additionally I would really like to see the use var when type is obvious relaxed. Because almost everywhere else tells me to not use var.

[Youssef1313]

Sometimes I find it hard, sometimes I don't. thinking about it, it is maybe harder than it was with var before.

It's not only about var, it's also about patterns like the following:

testfx/src/Platform/Microsoft.Testing.Extensions.VSTestBridge/ObjectModel/ObjectModelConverters.cs

Line 74 in aaa16e9

testNode.Properties.Add(new TestFileLocationProperty(testCase.CodeFilePath, new(new(testCase.LineNumber, -1), new(testCase.LineNumber, -1))));

which IMO should be really avoided.

[nohwnd]

I see. There I can still kind of guess what it is doing, but looking at the code without ide support or trying to quickly understand is definitely more difficult than if the whole type name was spelled out.