Github user dcelasun commented on a diff in the pull request:

    https://github.com/apache/thrift/pull/1507#discussion_r174812332
  
    --- Diff: lib/go/test/dontexportrwtest/compile_test.go ---
    @@ -29,10 +28,10 @@ import (
     func TestReadWriteMethodsArePrivate(t *testing.T) {
        // This will only compile if read/write methods exist
        s := NewTestStruct()
    -   fmt.Sprintf("%v", s.read)
    -   fmt.Sprintf("%v", s.write)
    +   _ = s.read
    --- End diff --
    
    The problem is `s.read` and `s.write` are functions and there just isn't 
any way to print functions as strings, so I had to find another way to use them 
without trying to stringify them.


---

Reply via email to