This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Re: Patch to change warn_write_strings to flag_const_strings


>>>>> Jim Wilson <wilson@cygnus.com> writes:

 > 	For C++, string constants are `const char[]' by default, so I'd
 > like to change the flag that controls this behavior in c-common.c, and
 > in any case it seems odd to control semantics with a warn_* flag.  What
 > do you think?

 > I agree, but it seems just as odd to have a -W option set a flag_* variable,
 > so I don't see that we have gained much unless you plan to do something else
 > like add a new -f option or change documentation.

The new -f option would be in the C++ frontend; we will have separate
-fconst-strings and -Wwrite-strings.  I thought it would be less confusing
to have the internal flags named appropriately.

Jason


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]