Skip to content

Mekhanik evgenii/1411 7 #2403

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

EvgeniiMekhanik
Copy link
Contributor

No description provided.

@EvgeniiMekhanik EvgeniiMekhanik requested review from const-t and krizhanovsky and removed request for const-t April 13, 2025 21:33
@krizhanovsky krizhanovsky removed their request for review April 14, 2025 12:15
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch 2 times, most recently from 6df4adf to 659ade7 Compare April 15, 2025 01:18
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch 2 times, most recently from 908e709 to 845ba21 Compare April 17, 2025 05:57
@EvgeniiMekhanik EvgeniiMekhanik marked this pull request as draft April 17, 2025 15:32
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch from 845ba21 to ac308cf Compare April 18, 2025 05:10
@EvgeniiMekhanik EvgeniiMekhanik marked this pull request as ready for review April 18, 2025 05:22
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch from ac308cf to 6154350 Compare April 18, 2025 14:16
According information from ChatGPT huffman encoding
gives compression benefit about 20 - 60 %. According
the article from the task #1411 huffman gives benefit
about 20 %. We decide to allocate extra 50 % space
during huffan decoding to prevent extra allocations
(make only one allocation).
@EvgeniiMekhanik EvgeniiMekhanik marked this pull request as draft May 2, 2025 20:12
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch from 06d356a to ee8b2c0 Compare May 2, 2025 20:12
- Do not allocate new memory during hpack decoding
  if some memory is already available. (It can be
  available because we allocate 150 % memory for
  each header).
- Move BUFFER_GET macros to hpack.c
- Some codestyle fixes according checkpatch.pl
@EvgeniiMekhanik EvgeniiMekhanik force-pushed the MekhanikEvgenii/1411-7 branch from ee8b2c0 to 466c5de Compare May 2, 2025 22:04
@EvgeniiMekhanik EvgeniiMekhanik marked this pull request as ready for review May 2, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant