On 03/10/15 10:44 -0600, Sandra Loosemore wrote:
Thanks for tackling this. I remember thinking that this section looked bit-rotted when I was reviewing the manual earlier this year. Your patch looks like a step in the right direction, but can I get you to fix a couple other things while you're at it?

First, I think the reference to ancient ld versions is confusing, and it would be better to rewrite that to emphasize that this is the default behavior on most targets. (I'd guess that anybody trying to use a recent GCC release with an ld version from 1996 is going to run into more critical blocking issues than this one.) Maybe something like:

"G++ implements the Borland model on targets where the linker supports it, including both ELF targets (such as GNU/Linux) and Microsoft Windows. Otherwise G++ implements neither automatic model."

Second, if "Do nothing" is now the recommended way to handle this, let's move that option to the front of the itemized list instead of the end. Also, I'm confused by the "pretend" here; can we just delete that sentence?

Yes, all good ideas, I'll incorporate these asap.

Reply via email to