Thrift
  1. Thrift
  2. THRIFT-477

remove extra methods generated for collections

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Invalid
    • Affects Version/s: 0.1
    • Fix Version/s: 0.2
    • Component/s: Java - Compiler
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      For some reason, the java generator produces a bunch of extra methods whenever you have a field of a collection type. For instance if you have a field called "my_list", then you will get a "get_my_list_size()" method in addition to the standard getter and setter.

      Personally I find these extra methods redundant, since all they save you is a pair of parentheses. I'd prefer to just get rid of them. Does anyone out there like or use these methods?

      1. thrift-477.patch
        3 kB
        Bryan Duxbury

        Activity

        Bryan Duxbury created issue -
        Bryan Duxbury made changes -
        Field Original Value New Value
        Attachment thrift-477.patch [ 12409417 ]
        Bryan Duxbury made changes -
        Assignee Bryan Duxbury [ bryanduxbury ]
        Bryan Duxbury made changes -
        Patch Info [Patch Available]
        Bryan Duxbury made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.2 [ 12313769 ]
        Resolution Fixed [ 1 ]
        Bryan Duxbury made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Bryan Duxbury [ bryanduxbury ]
        Todd Lipcon made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Invalid [ 6 ]
        Jake Farrell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Bryan Duxbury
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development