changeset 8241:db745339e54b quic

The ngx_quic_frame_len() function is not really needed.
author Vladimir Homutov <vl@nginx.com>
date Thu, 19 Mar 2020 14:59:55 +0300
parents 1f002206a59b
children 83a78cca8bce
files src/event/ngx_event_quic.c src/event/ngx_event_quic_transport.c src/event/ngx_event_quic_transport.h
diffstat 3 files changed, 1 insertions(+), 29 deletions(-) [+]
line wrap: on
line diff
--- a/src/event/ngx_event_quic.c
+++ b/src/event/ngx_event_quic.c
@@ -997,7 +997,7 @@ ngx_quic_output(ngx_connection_t *c)
         do {
             /* process same-level group of frames */
 
-            len += ngx_quic_frame_len(f);// TODO: handle overflow, max size
+            len += ngx_quic_create_frame(NULL, NULL, f);// TODO: handle overflow, max size
 
             f = f->next;
         } while (f && f->level == lvl);
--- a/src/event/ngx_event_quic_transport.c
+++ b/src/event/ngx_event_quic_transport.c
@@ -823,33 +823,6 @@ ngx_quic_create_frame(u_char *p, u_char 
 }
 
 
-size_t
-ngx_quic_frame_len(ngx_quic_frame_t *frame)
-{
-     switch (frame->type) {
-        case NGX_QUIC_FT_ACK:
-            return ngx_quic_create_ack(NULL, &frame->u.ack);
-        case NGX_QUIC_FT_CRYPTO:
-            return ngx_quic_create_crypto(NULL, &frame->u.crypto);
-
-        case NGX_QUIC_FT_STREAM0:
-        case NGX_QUIC_FT_STREAM1:
-        case NGX_QUIC_FT_STREAM2:
-        case NGX_QUIC_FT_STREAM3:
-        case NGX_QUIC_FT_STREAM4:
-        case NGX_QUIC_FT_STREAM5:
-        case NGX_QUIC_FT_STREAM6:
-        case NGX_QUIC_FT_STREAM7:
-            return ngx_quic_create_stream(NULL, &frame->u.stream);
-        case NGX_QUIC_FT_CONNECTION_CLOSE:
-            return ngx_quic_create_close(NULL, &frame->u.close);
-        default:
-            /* BUG: unsupported frame type generated */
-            return 0;
-     }
-}
-
-
 static size_t
 ngx_quic_create_ack(u_char *p, ngx_quic_ack_frame_t *ack)
 {
--- a/src/event/ngx_event_quic_transport.h
+++ b/src/event/ngx_event_quic_transport.h
@@ -200,6 +200,5 @@ ngx_int_t ngx_quic_parse_handshake_heade
 ssize_t ngx_quic_parse_frame(ngx_quic_header_t *pkt, u_char *start, u_char *end,
     ngx_quic_frame_t *frame);
 ssize_t ngx_quic_create_frame(u_char *p, u_char *end, ngx_quic_frame_t *f);
-size_t ngx_quic_frame_len(ngx_quic_frame_t *frame);
 
 #endif /* _NGX_EVENT_QUIC_WIRE_H_INCLUDED_ */