[commit] Correct style issues in xtensa-tdep.c

[Maxim Grigoriev <maxim@tensilica.com>]

[-- Attachment #1: Type: text/plain, Size: 918 bytes --]

I am surprised I missed it. Thanks for noticing this.
This patch corrects the style violation mentioned by Joel.

On 03/10/2011 10:45 PM, Joel Brobecker wrote:
>> 2011-03-10  Maxim Grigoriev<maxim2405@gmail.com>
>>
>> 	* xtensa-tdep.c (windowing_enabled): Remove inline attribute.
>> 	(xtensa_write_register, xtensa_read_register): Likewise.
>> 	(xtensa_hextochar): Removed.
>> 	(xtensa_init_reggroups): Replace xtensa_hextochar () by explicit code.
>>      
> Small comments, seen while scanning your patch...
>
>    
>> -static inline void warning_once ()
>> +static void warning_once ()
>>      
> The function name should be on the first column of the line.
> This is to facilitate the search of the function implementation
> (we can use:<<grep "^warning_once">>  for instance). Also,
> functions with no arguments should be declared with a "void"
> parameter. Thus:
>
> static void
> warning_once (void)
>
>    


[-- Attachment #2: Style.diff --]
[-- Type: text/plain, Size: 686 bytes --]

2011-03-11  Maxim Grigoriev  <maxim2405@gmail.com>

	* xtensa-tdep.c (warning_once): Correct style issues.

Index: gdb/xtensa-tdep.c
===================================================================
RCS file: /cvs/src/src/gdb/xtensa-tdep.c,v
retrieving revision 1.54
diff -u -r1.54 xtensa-tdep.c
--- gdb/xtensa-tdep.c	11 Mar 2011 02:32:31 -0000	1.54
+++ gdb/xtensa-tdep.c	11 Mar 2011 18:25:19 -0000
@@ -1421,7 +1421,8 @@
 /* Report a problem with prologue analysis while doing backtracing.
    But, do it only once to avoid annoyng repeated messages.  */
 
-static void warning_once ()
+static void
+warning_once (void)
 {
   if (xtensa_session_once_reported == 0)
     warning (_("\