facebook/zstd

Refactor ZSTD_estimateCCtxSize_usingCCtxParams_internal()

Open

#3.404 geöffnet am 28. Dez. 2022

Auf GitHub ansehen
 (5 Kommentare) (1 Reaktion) (1 zugewiesene Person)C (27.100 Stars) (2.481 Forks)batch import
enhancementgood first issuelong-term

Beschreibung

ZSTD_estimateCCtxSize_usingCCtxParams_internal() currently takes a long list of parameters. Some (all?) of the parameters are extracted from external ZSTD_CCtx_params objects and passed in one by one. It would be cleaner to reduce the number of parameters by directly passing in a ZSTD_CCtx_params const*.

This should be an easy refactor. Code pointer: https://github.com/facebook/zstd/blob/90597d78eadadae7acf3633ab4d81ef7c4e6057f/lib/compress/zstd_compress.c#L1488-L1495

Contributor Guide

Tech Stack
c
Domain
backend
Issue Type
refactor
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
blocked
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
C programmingbasic understanding of zstd
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Modify the function signature of ZSTD estimateCCtxSize usingCCtxParams internal in lib/compress/zstd compress.c (line 1488) to accept a const pointer to ZSTD CCtx params instead of the individual parameters. Then update all callers of this function to pass the pointer. There may be internal and external callers to consider. Ensure the change is consistent with the existing code style and passes tests.