Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-3149

Make ReadFieldN methods in generated Go code private

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 0.9.2
    • 0.9.3
    • Go - Compiler
    • None

    Description

      The go thrift compiler generates ReadFieldN and writeFieldN methods for every field of a struct. E.g.:

      func (p *Statistics) ReadField1(iprot thrift.TProtocol) error {
      ...
      }
      
      func (p *Statistics) writeField1(oprot thrift.TProtocol) (err error) {
      ...
      }
      

      ReadField1, ReadField2, etc are not very descriptive names and shouldn't be part of public API of a generated package (like writeFieldN).

      Attachments

        Issue Links

          Activity

            People

              k.shaposhnikov@gmail.com Konstantin Shaposhnikov
              k.shaposhnikov@gmail.com Konstantin Shaposhnikov
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: