[patch] Hide the caret for -Wstack-usage

Eric Botcazou ebotcazou@adacore.com
Sat Jul 12 10:41:00 GMT 2014


> Your patch hides the caret but the location still points to the
> closing brace (so if you use an IDE like emacs this is where you will
> go).
> 
> Is there a location for the definition? Could we store/pass one to this
> point?
> 
> Otherwise, would it be appropriate to point to the declaration of the
> function? After all, this is what -fstack-usage seems to do.

Good idea, let's use a consistent location, revised patch attached, thanks.


2014-07-12  Eric Botcazou  <ebotcazou@adacore.com>

	* toplev.c (output_stack_usage): Adjust the location and temporarily
	hide the caret.


2014-07-12  Eric Botcazou  <ebotcazou@adacore.com>

	* gcc.dg/stack-usage-2.c: Adjust.


-- 
Eric Botcazou
-------------- next part --------------
A non-text attachment was scrubbed...
Name: p.diff
Type: text/x-patch
Size: 2379 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20140712/9c79c667/attachment.bin>


More information about the Gcc-patches mailing list