fix as commented: use embedding to get rid of delegations

Signed-off-by: Sandor Szücs <sandor.szuecs@zalando.de>

tracing/tracers/basic/basic.go

@@ -17,9 +17,9 @@ type CloseableTracer interface {
 }
 
 type basicTracer struct {
-	tracer opentracing.Tracer
-	quit   chan struct{}
-	once   sync.Once
+	opentracing.Tracer
+	quit chan struct{}
+	once sync.Once
 }
 
 func InitTracer(opts []string) (CloseableTracer, error) {
@@ -71,14 +71,14 @@ func InitTracer(opts []string) (CloseableTracer, error) {
 
 	quit := make(chan struct{})
 	bt := &basicTracer{
-		tracer: basic.NewWithOptions(basic.Options{
+		basic.NewWithOptions(basic.Options{
 			DropAllLogs:    dropAllLogs,
 			ShouldSample:   func(traceID uint64) bool { return traceID%sampleModulo == 0 },
 			MaxLogsPerSpan: maxLogsPerSpan,
 			Recorder:       recorder,
 		}),
-		quit: quit,
-		once: sync.Once{},
+		quit,
+		sync.Once{},
 	}
 
 	go func() {
@@ -111,18 +111,6 @@ func invalidArg(opt string, err error) error {
 	return fmt.Errorf("invalid argument for %s option: %s", opt, err)
 }
 
-func (bt *basicTracer) StartSpan(operationName string, opts ...opentracing.StartSpanOption) opentracing.Span {
-	return bt.tracer.StartSpan(operationName, opts...)
-}
-
-func (bt *basicTracer) Inject(sm opentracing.SpanContext, format interface{}, carrier interface{}) error {
-	return bt.tracer.Inject(sm, format, carrier)
-}
-
-func (bt *basicTracer) Extract(format interface{}, carrier interface{}) (opentracing.SpanContext, error) {
-	return bt.tracer.Extract(format, carrier)
-}
-
 func (bt *basicTracer) Close() {
 	bt.once.Do(func() {
 		close(bt.quit)