jnferguson / hpack-rfc7541

header-only c++ implementation of a HTTPv2 HPACK/RFC7541 encoder /decoder
BSD 2-Clause "Simplified" License
19 stars 7 forks source link

typo? #5

Open ghost opened 4 years ago

ghost commented 4 years ago

if ( 0x20 == ( itr 0xE0 ) ) { // 6.3 Dynamic Table update

I think that should've been:

if ( 0x20 == ( *itr & 0xE0 ) ) { // 6.3 Dynamic Table update

Also: } else if ( index > predefined_headers.size() && index < predefined_headers.size() + m_queue.size() )

i think that should've been:

} else if ( index >= predefined_headers.size() && index < predefined_headers.size() + m_queue.size() )