Removed target rtp_to_text

webrtc/modules/audio_coding/BUILD.gn

 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 import("../../build/webrtc.gni")
 import("audio_coding.gni")
@@ skipping 1191 matching lines @@
     ]
 
     if (is_win) {
       cflags = [
         # Disable warnings to enable Win64 build, issue 1323.
         "/wd4267",  # size_t to int truncation
       ]
     }
   }
 
+  executable("rtp_to_text") {

[aleloi, 2016/08/01 15:26:33]

This seems unused or deprecated. There is a 'return -1' in the main loop, and it
uses DataLog which is not used anywhere else and has no implementation. Can I
get confirmation on this and should we skip this target?

[kjellander_webrtc, 2016/08/01 19:03:43]

Let's see what Henrik L has to say. I believe he added it.

+    testonly = true
+
+    sources = [
+      "neteq/test/rtp_to_text.cc",
+    ]
+
+    deps = [
+      ":neteq_test_tools",
+      "../../system_wrappers",
+    ]
+  }
+
   executable("rtp_analyze") {
     testonly = true
 
     sources = [
       "neteq/tools/rtp_analyze.cc",
     ]
 
     deps = [
       ":neteq",
       ":neteq_unittest_tools",
       ":pcm16b",
       "../../system_wrappers:system_wrappers_default",
       "//testing/gtest",
       "//third_party/gflags:gflags",
     ]
 
     if (is_clang) {
       # Suppress warnings from Chrome's Clang plugins.
       # See http://code.google.com/p/webrtc/issues/detail?id=163 for details.
       configs -= [ "//build/config/clang:find_bad_constructs" ]
     }
   }
 }