------- Additional Comments From nickc at redhat dot com 2009-01-06 16:36 ------- Hi Sebastian,
The patch looks good - thanks very much for persevering with this. In answer to your questions: * Yes stat_alloc() is the right function to use. * Quoting names like this is preferred. * You are correct, the prototype for lang_memory_region_default is redundant. * There is no separate documentation for writing testcases. But it is relatively simple to do by taking an already existing testcase (or two) and modifying them to suit your needs. So, in order to accept this patch, or rather an extended version of it, we will need: * A copyright assignment on file with the FSF for binutils contributions. * Documentation of the new feature in ld/ld.texinfo plus a one line description of it in ld/NEWS. * A testcase or two to make sure that it works, and continues to work as changes are made to the linker in the future. Cheers Nick -- http://sourceware.org/bugzilla/show_bug.cgi?id=7031 ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ bug-binutils mailing list bug-binutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-binutils