adjust continuation frame header length#14245
Conversation
|
I would argue we should ignore the Revapi failure here as at worse it will just use the old value and so allocate 1 extra byte until things are re-compiled. @chrisvest WDYT ? |
|
@normanmaurer I agree. Revapi is flagging it because such a change could be problematic, but this case should be fine as you say. |
okay，i fix it |
Motivation: Padding was removed from CONTINUATION frame in http2-spec, at the same time, the continuation frame header should be adjusted, which can also to reduce the allocation of bytes. Modifications: Adjust number of bytes Result: Reduce num of bytes that are allocated
Motivation: Padding was removed from CONTINUATION frame in http2-spec, at the same time, the continuation frame header should be adjusted, which can also to reduce the allocation of bytes. Modifications: Adjust number of bytes Result: Reduce num of bytes that are allocated
|
@CLFutureX Thanks a lot! |
Motivation:
Padding was removed from CONTINUATION frame in http2-spec,
at the same time , the continuation frame header should be adjusted,
which can also to reduce the allocation of bytes.