Page MenuHomePhabricator

[yaml2obj] - Implement the "Offset" property for the Fill Chunk.

Authored by grimar on Tue, May 19, 1:18 AM.



Similar to a regular section chunk, a Fill should have this property.
This patch implements it.

Diff Detail

Event Timeline

grimar created this revision.Tue, May 19, 1:18 AM
Herald added a project: Restricted Project. · View Herald Transcript
MaskRay accepted this revision.Tue, May 19, 10:02 AM

Looks great!

This revision is now accepted and ready to land.Tue, May 19, 10:02 AM
jhenderson added inline comments.Wed, May 20, 1:35 AM

arbitraty -> arbitrary


I'm not sure we really need to check the section header table offset, as the position of .bar is sufficient to show the fill impacts the offset calculations.

grimar updated this revision to Diff 265180.Wed, May 20, 2:33 AM
grimar marked 3 inline comments as done.
  • Addressed review comments.

Probably you're right. Not sure why I've added this line.

This revision was automatically updated to reflect the committed changes.