m4b / goblin

An impish, cross-platform binary parsing crate, written in Rust
MIT License
1.18k stars 158 forks source link

Documented the pe::header::DosHeader. #393

Closed JohnScience closed 7 months ago

JohnScience commented 7 months ago

I've noticed that there was little to no documentation of the DOS header, so I decided to add it. I've provided sources for all the claims that are made in the docs.

JohnScience commented 7 months ago

@m4b

m4b commented 7 months ago

I have to ask, was the documentation generated using some form of automation or with an AI assistant?

m4b commented 7 months ago

release note: non-breaking

JohnScience commented 7 months ago

I have to ask, was the documentation generated using some form of automation or with an AI assistant?

For the most part, the documentation was written manually with the assistance of GitHub Copilot. There was a small exception, though. I haven't found a website with decent explanation of one topic of Program Segment Prefix (PSP) so I fact-checked the paragraph generated by ChatGPT and used it to make the part about PSP easier to understand.