Skip to content

Conversation

@hxzhao527
Copy link

@hxzhao527 hxzhao527 commented Aug 22, 2024

@hxzhao527 hxzhao527 changed the title feat: Add support for Android compressed rel/rela sections WIP: Add support for Android compressed rel/rela sections Aug 23, 2024
@hxzhao527 hxzhao527 force-pushed the feature/android_relocation branch 3 times, most recently from 9769970 to 17036f2 Compare August 25, 2024 07:41
@hxzhao527 hxzhao527 changed the title WIP: Add support for Android compressed rel/rela sections Add support for Android compressed rel/rela sections Aug 25, 2024
@hxzhao527 hxzhao527 force-pushed the feature/android_relocation branch from 17036f2 to c02757b Compare August 25, 2024 07:49
@hxzhao527
Copy link
Author

@cole14 could you review this? thanks

@cole14
Copy link
Owner

cole14 commented Aug 28, 2024

Thanks for the PR! I can probably take a deeper look some time in the next few days

}

impl<'data> Iterator for ParsingIterator<'data, false> {
type Item = Result<Rel, ParseError>;
Copy link
Author

Choose a reason for hiding this comment

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

here, a different return value style from other iterators in the project.

when &[8] got unexpected eof, parsing .rel got rela and etc will get Err(...)

@hxzhao527 hxzhao527 force-pushed the feature/android_relocation branch from c02757b to 27e7827 Compare December 5, 2024 03:18
@hxzhao527 hxzhao527 force-pushed the feature/android_relocation branch 2 times, most recently from 7c2b409 to 6afb4e4 Compare March 16, 2025 09:31
@DCNick3
Copy link

DCNick3 commented Aug 15, 2025

It would be nice to see this supported

@hxzhao527 hxzhao527 force-pushed the feature/android_relocation branch from 6afb4e4 to bca3bb3 Compare August 15, 2025 13:10
@qwq233
Copy link

qwq233 commented Oct 30, 2025

kinda curious when will this pr merge to the main branch. quite need this for my personal project.

or is there other library that can parse ELF easily?

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.

4 participants