Skip to content

Make Tag::encoded_length public#596

Closed
facutuesca wants to merge 1 commit intoalex:mainfrom
trail-of-forks:ft/public-tag-method
Closed

Make Tag::encoded_length public#596
facutuesca wants to merge 1 commit intoalex:mainfrom
trail-of-forks:ft/public-tag-method

Conversation

@facutuesca
Copy link
Contributor

@facutuesca facutuesca commented Feb 14, 2026

This PR makes Tag::encoded_length public.

With this, users who want to get the tag bytes from a Tlv can use Tlv::full_data() and Tag::encoded_length() in order to do so.

@facutuesca facutuesca changed the title Make Tag::encoded_length public and prepare for releasing 0.24.0 Make Tag::encoded_length public Feb 26, 2026
@facutuesca facutuesca changed the title Make Tag::encoded_length public Make Tag::write_to private and Tag::encoded_length public Feb 26, 2026
@facutuesca
Copy link
Contributor Author

Rebased and removed the version bump

Signed-off-by: Facundo Tuesca <facundo.tuesca@trailofbits.com>
@facutuesca facutuesca force-pushed the ft/public-tag-method branch from ace7326 to 06ea97e Compare March 18, 2026 01:31
@facutuesca facutuesca changed the title Make Tag::write_to private and Tag::encoded_length public Make Tag::encoded_length public Mar 18, 2026
@facutuesca
Copy link
Contributor Author

Closing in favor of #608

@facutuesca facutuesca closed this Mar 21, 2026
@facutuesca facutuesca deleted the ft/public-tag-method branch March 21, 2026 11:32
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