Remove another TODO - no need to set "id" field of stop markers

Test plan: tested on our price service

git-svn-id: https://svn.apache.org/repos/asf/incubator/thrift/trunk@666457 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/lib/alterl/src/thrift_binary_protocol.erl b/lib/alterl/src/thrift_binary_protocol.erl
index c84701e..804e0ad 100644 (file)
--- a/lib/alterl/src/thrift_binary_protocol.erl
+++ b/lib/alterl/src/thrift_binary_protocol.erl
@@ -190,8 +190,7 @@ read(This, struct_end) -> ok;
 read(This, field_begin) ->
     case read(This, byte) of
         {ok, Type = ?tType_STOP} ->
-            #protocol_field_begin{type = Type,
-                                  id = 0}; % TODO(todd) 0 or undefined?
+            #protocol_field_begin{type = Type};
         {ok, Type} ->
             {ok, Id} = read(This, i16),
             #protocol_field_begin{type = Type,