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

Compiled ruby code generates warning if field with name "fields" is present

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 0.9
    • None
    • Ruby - Compiler
    • None

    Description

      The compiler generates ruby code which uses a constant called FIELDS. It also converts each thrift field into a constant. So a field called "fields" will end up generating a constant with same name, producing the warning.

      $ cat foo.thrift
      struct Foo {
        1: list<string> fields;
      }
      $ thrift --gen rb -out . foo.thrift
      $ irb
      > require './foo_types'
      /Users/ben/code/eqm/ruby-storm-connector/foo_types.rb:13: warning: already initialized constant FIELDS
       => true
      

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            bensymonds Ben Symonds

            Dates

              Created:
              Updated:

              Slack

                Issue deployment