@vsabella I guess we should just remove it completely. 

@carl-mastrangelo @ejona86 @Scottmitch thoughts ? Also @mosesn @bryce-anderson WDYT ? Looks like dead code to me and `Http2CodecUtil` is still tagged `@UnstableApi` so removing it seems like the right move. @bryce-anderson sounds good... Will try to find time to write a patch or if you want you could also if you have time 👍  I can take it. Unless it is actually in use it should be trivial to remove. Thanks that would be awesome

> Am 06.06.2018 um 14:28 schrieb Bryce Anderson <notifications@github.com>:
> 
> I can take it. Unless it is actually in use it should be trivial to remove.
> 
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub, or mute the thread.
 Thanks @bryce-anderson 
Http2CodecUtil.java - emptyPingBuf() is no longer needed (as writePing takes a long)
https://github.com/netty/netty/blob/b192bf12ad2af92bba0f32c9d3127b1192e54670/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java#L177


In https://github.com/netty/netty/commit/501662a77f8166382eefd18a8620983d27f59d4a - emptyPing data changed to long (instead of buffer data)

This should be updated.
codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java
codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java
diff --git a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java
index 317ee48063d..7ebc8fd4888 100644
--- a/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java
+++ b/codec-http2/src/main/java/io/netty/handler/codec/http2/Http2CodecUtil.java
@@ -67,9 +67,6 @@ public final class Http2CodecUtil {
     private static final ByteBuf CONNECTION_PREFACE =
             unreleasableBuffer(directBuffer(24).writeBytes("PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n".getBytes(UTF_8)))
                     .asReadOnly();
-    private static final ByteBuf EMPTY_PING =
-            unreleasableBuffer(directBuffer(PING_FRAME_PAYLOAD_LENGTH).writeZero(PING_FRAME_PAYLOAD_LENGTH))
-                    .asReadOnly();
 
     private static final int MAX_PADDING_LENGTH_LENGTH = 1;
     public static final int DATA_FRAME_HEADER_LENGTH = FRAME_HEADER_LENGTH + MAX_PADDING_LENGTH_LENGTH;
@@ -169,14 +166,6 @@ public static ByteBuf connectionPrefaceBuf() {
         return CONNECTION_PREFACE.retainedDuplicate();
     }
 
-    /**
-     * Returns a buffer filled with all zeros that is the appropriate length for a PING frame.
-     */
-    public static ByteBuf emptyPingBuf() {
-        // Return a duplicate so that modifications to the reader index will not affect the original buffer.
-        return EMPTY_PING.retainedDuplicate();
-    }
-
     /**
      * Iteratively looks through the causality chain for the given exception and returns the first
      * {@link Http2Exception} or {@code null} if none.
