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
--- a/lib/alterl/src/thrift_binary_protocol.erl
+++ b/lib/alterl/src/thrift_binary_protocol.erl
@@ -190,8 +190,7 @@
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,