Merge pull request #22 from tcolgate/master

basvanbeek · web-flow · commit 3dc21067bf69 · 2016-09-01T14:05:34.000+02:00
Add NewSpanEventListener as an option.
diff --git a/tracer.go b/tracer.go
@@ -173,6 +173,14 @@ func EnableSpanPool(val bool) TracerOption {
 	}
 }
 
+// NewSpanEventListener option
+func NewSpanEventListener(f func() func(SpanEvent)) TracerOption {
+	return func(opts *TracerOptions) error {
+		opts.newSpanEventListener = f
+		return nil
+	}
+}
+
 // NewTracer creates a new OpenTracing compatible Zipkin Tracer.
 func NewTracer(recorder SpanRecorder, options ...TracerOption) (opentracing.Tracer, error) {
 	opts := &TracerOptions{

Original file line number	Diff line number	Diff line change
`@@ -173,6 +173,14 @@ func EnableSpanPool(val bool) TracerOption {`
`173`	`173`	`}`
`174`	`174`	`}`
`175`	`175`
	`176`	`+// NewSpanEventListener option`
	`177`	`+func NewSpanEventListener(f func() func(SpanEvent)) TracerOption {`
	`178`	`+ return func(opts *TracerOptions) error {`
	`179`	`+ opts.newSpanEventListener = f`
	`180`	`+ return nil`
	`181`	`+ }`
	`182`	`+}`
	`183`	`+`
`176`	`184`	`// NewTracer creates a new OpenTracing compatible Zipkin Tracer.`
`177`	`185`	`func NewTracer(recorder SpanRecorder, options ...TracerOption) (opentracing.Tracer, error) {`
`178`	`186`	`opts := &TracerOptions{`