Remove unnecessary nil check

This member is never nil, unless the Exporter is created like &Exporter{}, which is not a thing we support anyway.
open-telemetry · Nov 20, 2020 · 4f2b83b · 4f2b83b
1 parent ab26835
commit 4f2b83b
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/exporters/otlp/otlp.go b/exporters/otlp/otlp.go
@@ -143,10 +143,8 @@ func (e *Exporter) Shutdown(ctx context.Context) error {
 	var err error
 
 	e.stopOnce.Do(func() {
-		if cc != nil {
-			// Clean things up before checking this error.
-			err = cc.shutdown(ctx)
-		}
+		// Clean things up before checking this error.
+		err = cc.shutdown(ctx)
 
 		// At this point we can change the state variable started
 		e.mu.Lock()