PR feedback, avoid null-checking for activity

Kielek · Kielek · commit 1aaea5295b0b · 2024-05-14T08:43:44.000+02:00
it is always not-null (checked in parent method)
diff --git a/src/OpenTelemetry.Instrumentation.AspNetCore/Implementation/HttpInListener.cs b/src/OpenTelemetry.Instrumentation.AspNetCore/Implementation/HttpInListener.cs
@@ -60,12 +60,7 @@ public HttpInListener(AspNetCoreTraceInstrumentationOptions options)
 
     public override void OnEventWritten(string name, object? payload)
     {
-        var activity = Activity.Current;
-
-        if (activity == null)
-        {
-            return;
-        }
+        var activity = Activity.Current!;
 
         switch (name)
         {

Original file line number	Diff line number	Diff line change
`@@ -60,12 +60,7 @@ public HttpInListener(AspNetCoreTraceInstrumentationOptions options)`
`60`	`60`
`61`	`61`	`public override void OnEventWritten(string name, object? payload)`
`62`	`62`	`{`
`63`		`- var activity = Activity.Current;`
`64`		`-`
`65`		`- if (activity == null)`
`66`		`- {`
`67`		`- return;`
`68`		`- }`
	`63`	`+ var activity = Activity.Current!;`
`69`	`64`
`70`	`65`	`switch (name)`
`71`	`66`	`{`