[patch PE-COFF]: Some minor nits

Kai Tietz Kai.Tietz@onevision.com
Fri Mar 13 11:02:00 GMT 2009


Hello,

Some minor nits I found in coffcode.h. One is that debugging sections have 
to have readonly flag set (abstracted by the section flags of '.debug' in 
pe-coff spec).
Secondly a clean-up about NOREAD sections. I introduced a helper flag for 
section's flags to indicated, that a section has no memory flag set,
I tested this patch for i686-pc-cygwin and for x86_64-pc-ming32 and saw no 
regressions on testrun.

ChangeLog
2009-03-13  Kai Tietz  <kai.tietz@onevision.com>

        * bfd-in2.h: Regenerated.
        * coffcode.h (sec_to_styp_flags): For pe-coff add SEC_READONLY
        for debugging sections and map memory read/write dependent on
        SEC_COFF_NOREAD.
        (styp_to_sec_flags): Set SEC_COFF_NOREAD for sections
        without memory read flags set.
        * section.c: Add SEC_COFF_NOREAD to section flags.

Is this patch ok for apply?
 
Cheers,
Kai



|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: noread.diff
Type: application/octet-stream
Size: 3555 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20090313/ee193ecc/attachment.obj>


More information about the Binutils mailing list