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]
Other format: [Raw text]

Re: [PATCH] Fix PR/8268: implement compile time array subscript checking


On 16/01/07, Dirk Mueller <dmuell@gmx.net> wrote:
On Tuesday, 16. January 2007 15:16, Manuel López-Ibáñez wrote:

> Also, the placemente is far from ideal, the option is not enabled by
> Wall (despite of what your Changelog says)

I forgot this particular hunk in the patch I mailed. I've resorted the invoke
entry to be in alphabetical order and added the -Wno-array-bounds opindex.


Thanks! Then, it is typical to add the sentence "This warning is enabled by -Wall." after the description of each warning that is enabled by -Wall. See -Wtrigraphs, -Wuninitialized, etc.

Also, I am not sure whether you need testcases for "testing that is
enabled by -Wall" and "testing that is disabled with -Wall
-Wno-array-bounds". Let's see what diagnostics maintainers say...

Cheers,

Manuel.

PS: I guess that the new option needs to be documented in
http://gcc.gnu.org/gcc-4.3/changes.html once you have finished with
it.


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