Should NoopSpanBuilder still extend NoopSpanContext?

[cwe1ss]

Hi! I'm working on the C# version of opentracing-noop and I've seen that NoopSpanBuilder extends NoopSpanContext.

Is there still any use for this or is this no longer necessary due to #55, #121 ?

[carlosalberto]

Hey @cwe1ss

It looks to me like #121 included a lot of changes after refactoring (19 files in total), and my bet is that NoopSpanBuilder was forgotten all along (I think this because in the PR/Issue there's no mention of NoopSpanBuilder).

I tried updating the code to remove that implements SpanContext + minor tuning, and everything works just fine, so I could finish the patch up and send it as a final fix piece.

Does it make sense to you guys? @yurishkuro @bhs (author of #121) You may remember it better ;)

[bhs]

@carlosalberto @cwe1ss sorry for the delay! I don't know why that inheritance relationship would still be necessary. If everything compiles without it, we should def remove. Thanks for asking.