Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

Refactor - Remove UTF-8 from ELF paser #469

Merged
merged 1 commit into from
May 31, 2023
Merged

Conversation

Lichtso
Copy link

@Lichtso Lichtso commented May 31, 2023

No description provided.

@Lichtso Lichtso requested a review from alessandrod May 31, 2023 09:25
src/elf.rs Outdated Show resolved Hide resolved
Copy link

@alessandrod alessandrod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine. It does change behavior for the new parser, but since that's off it's not a consensus concern.

@Lichtso Lichtso force-pushed the refactor/remove_utf8_in_elf branch from d15af49 to 7f9936e Compare May 31, 2023 19:20
@Lichtso Lichtso requested a review from alessandrod May 31, 2023 19:20
@Lichtso Lichtso force-pushed the refactor/remove_utf8_in_elf branch from 7f9936e to 6bf7ff1 Compare May 31, 2023 19:50
@Lichtso Lichtso merged commit 256b617 into main May 31, 2023
@Lichtso Lichtso deleted the refactor/remove_utf8_in_elf branch May 31, 2023 20:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants