Uploaded image for project: 'Go Driver'
  1. Go Driver
  2. GODRIVER-2722

Remove unnecessary interfaces from the merged "bson" package

XMLWordPrintableJSON

    • Type: Icon: Improvement Improvement
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 2.0.0
    • Affects Version/s: None
    • Component/s: BSON
    • None
    • Major Change
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      After merging the bson packages into one package, there are a bunch of exported interfaces. Some of them seem like they no longer need to be exported and some of them seem like duplicates.

      The full list of interfaces is:

      type ArrayReader
type ArrayWriter
type BytesReader
type BytesWriter
type Codec
type CodecZeroer
type DocumentReader
type DocumentWriter
type KeyMarshaler
type KeyUnmarshaler
type Marshaler
type Proxy
type Unmarshaler
type ValueDecoder
type ValueEncoder
type ValueMarshaler
type ValueReader
type ValueUnmarshaler
type ValueWriter
type ValueWriterFlusher
type Zeroer

      Audit that list of interfaces and remove or un-export any that don't need to be part of the public API.

      Definition of done:

      • Audit the list of exported interfaces in the merged bson package and remove or un-export any that don't need to be part of the public API.

            Assignee:
            qingyang.hu@mongodb.com Qingyang Hu
            Reporter:
            matt.dale@mongodb.com Matt Dale
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: