Remove unnecessary parentheses in declarations.
Commit Message
Hello,
Due to a new warning introduced to GCC last Thursday (commit a94975e57),
building binutils/gdb with upstream gcc fails. Therefore, I prepared a
patch to avoid triggering this new warning. Please review the patch and
apply if legitimate.
Thanks,
Egeyar
Comments
On 10/10/2017 08:17 PM, Egeyar Bagcioglu wrote:
> Hello,
>
> Due to a new warning introduced to GCC last Thursday (commit a94975e57),
> building binutils/gdb with upstream gcc fails. Therefore, I prepared a
> patch to avoid triggering this new warning. Please review the patch and
> apply if legitimate.
Merged. Thanks!
Pedro Alves
From 988d097e561d70b3c530a913e26f839fcbbcc467 Mon Sep 17 00:00:00 2001
From: Egeyar Bagcioglu <egeyar.bagcioglu@oracle.com>
Date: Tue, 10 Oct 2017 03:48:22 -0700
Subject: [PATCH] Remove unnecessary parentheses in declarations.
GCC commit a94975e57 introduces new warnings "unnecessary parentheses
in the declaration of ...". These cause the build of binutils and gdb
to fail. This patch removes those parentheses for a successful build.
2017-10-10 Egeyar Bagcioglu <egeyar.bagcioglu@oracle.com>
* macrotab.h (macro_lookup_inclusion): Remove unnecessary
parentheses in the declaration.
(macro_lookup_inclusion): Likewise.
(macro_lookup_definition): Likewise
* p-lang.h (pascal_builtin_types): Likewise.
* tui/tui-data.c (tui_win_list): Likewise.
* tui/tui-data.h (tui_win_list): Likewise.
* utils.h (make_cleanup_free_section_addr_info): Likewise.
---
gdb/macrotab.h | 12 ++++++------
gdb/p-lang.h | 2 +-
gdb/tui/tui-data.c | 2 +-
gdb/tui/tui-data.h | 2 +-
gdb/utils.h | 4 ++--
5 files changed, 11 insertions(+), 11 deletions(-)
@@ -230,9 +230,9 @@ void macro_define_special (struct macro_table *table);
path. e.g., `stdio.h', not `/usr/include/stdio.h'. If NAME
appears more than once in the inclusion tree, return the
least-nested inclusion --- the one closest to the main source file. */
-struct macro_source_file *(macro_lookup_inclusion
+struct macro_source_file *macro_lookup_inclusion
(struct macro_source_file *source,
- const char *name));
+ const char *name);
/* Record an object-like #definition (i.e., one with no parameter list).
@@ -315,9 +315,9 @@ struct macro_definition
effect at the end of the file. The macro table owns the structure;
the caller need not free it. Return zero if NAME is not #defined
at that point. */
-struct macro_definition *(macro_lookup_definition
+struct macro_definition *macro_lookup_definition
(struct macro_source_file *source,
- int line, const char *name));
+ int line, const char *name);
/* Return the source location of the definition for NAME in scope at
@@ -325,11 +325,11 @@ struct macro_definition *(macro_lookup_definition
number of the definition, and return a source file structure for
the file. Return zero if NAME has no definition in scope at that
point, and leave *DEFINITION_LINE unchanged. */
-struct macro_source_file *(macro_definition_location
+struct macro_source_file *macro_definition_location
(struct macro_source_file *source,
int line,
const char *name,
- int *definition_line));
+ int *definition_line);
/* Prototype for a callback callable when walking a macro table. NAME
is the name of the macro, and DEFINITION is the definition. SOURCE
@@ -59,7 +59,7 @@ extern void pascal_printstr (struct ui_file *, struct type *, const gdb_byte *,
unsigned int, const char *, int,
const struct value_print_options *);
-extern struct type **const (pascal_builtin_types[]);
+extern struct type **const pascal_builtin_types[];
/* These are in p-typeprint.c: */
@@ -29,7 +29,7 @@
/****************************
** GLOBAL DECLARATIONS
****************************/
-struct tui_win_info *(tui_win_list[MAX_MAJOR_WINDOWS]);
+struct tui_win_info *tui_win_list[MAX_MAJOR_WINDOWS];
/***************************
** Private data
@@ -293,7 +293,7 @@ extern void tui_set_win_highlight (struct tui_win_info *win_info,
/* Global Data. */
-extern struct tui_win_info *(tui_win_list[MAX_MAJOR_WINDOWS]);
+extern struct tui_win_info *tui_win_list[MAX_MAJOR_WINDOWS];
#define TUI_SRC_WIN tui_win_list[SRC_WIN]
#define TUI_DISASM_WIN tui_win_list[DISASSEM_WIN]
@@ -206,8 +206,8 @@ private:
/* Cleanup utilities. */
struct section_addr_info;
-extern struct cleanup *(make_cleanup_free_section_addr_info
- (struct section_addr_info *));
+extern struct cleanup *make_cleanup_free_section_addr_info
+ (struct section_addr_info *);
/* For make_cleanup_close see common/filestuff.h. */
--
1.9.1