Sergey Teplyakov issues

Results 86 issues of


Sergey Teplyakov

Detect that try/catch is used for a single method that is an iterator block

Iterator blocks never throw.

Warn if the result of `Task.Run` (or `Task.Factory.StartNew`) is not observed

Not sure I can think of any cases when this is a good idea. Any errors in this code will be unobserved.

Do not warn about a hidden copy if an extension method is used that causes boxing

In this case, the warning should be different and it should be covered by the memory analyzer.

Warn if a non-readonly variable is reassigned after passing it as an 'in'-argument?

Potentially the following code could be unsafe: ```csharp using System; public class C { public void M() { int n = 42; Foo(n); Action a = () => {Console.WriteLine(n); n++;};...

Warn that `if (x is var y)` is always true

Warn for value types being used in the Monitor.Enter/Exit

Keep a list of well-known mutable structs and warn if they're used with readonly modifier

Enumerators for BCL collections (and any struct-based iterators) SpinLock Analyze the BCL to find other instances.

Warn if DebuggerDisplayAttribute points to a method but doesn't have an invocation

This is the case: ```csharp [DebuggerDisplay("{Foo,nq}") class FooBar { private string Foo() {return string.Empty;} } ```

Warn on covarient array conversion

Analysis that will warn for self-assignment

It could sounds very simple and unlikely to happen, but I just got the situation when I've assigned property to itself! And as far as I know, there was even...