From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 7246 invoked by alias); 19 Apr 2011 10:12:33 -0000 Received: (qmail 7237 invoked by uid 22791); 19 Apr 2011 10:12:31 -0000 X-SWARE-Spam-Status: No, hits=-1.4 required=5.0 tests=AWL,BAYES_00,MSGID_MULTIPLE_AT,TW_XZ X-Spam-Check-By: sourceware.org Received: from mailhost.u-strasbg.fr (HELO mailhost.u-strasbg.fr) (130.79.200.154) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Tue, 19 Apr 2011 10:12:17 +0000 Received: from md1.u-strasbg.fr (md1.u-strasbg.fr [IPv6:2001:660:2402::186]) by mailhost.u-strasbg.fr (8.14.3/jtpda-5.5pre1) with ESMTP id p3JACEI9069229 for ; Tue, 19 Apr 2011 12:12:14 +0200 (CEST) (envelope-from pierre.muller@ics-cnrs.unistra.fr) Received: from mailserver.u-strasbg.fr (ms4.u-strasbg.fr [130.79.204.13]) by md1.u-strasbg.fr (8.14.4/jtpda-5.5pre1) with ESMTP id p3JACE8l000515 for ; Tue, 19 Apr 2011 12:12:14 +0200 (CEST) (envelope-from pierre.muller@ics-cnrs.unistra.fr) Received: from E6510Muller (gw-ics.u-strasbg.fr [130.79.210.225]) (user=mullerp mech=LOGIN) by mailserver.u-strasbg.fr (8.14.4/jtpda-5.5pre1) with ESMTP id p3JACENo031896 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO) for ; Tue, 19 Apr 2011 12:12:14 +0200 (CEST) (envelope-from pierre.muller@ics-cnrs.unistra.fr) From: "Pierre Muller" To: Subject: [RFA] Lowercase static function names that use capital letters. Date: Tue, 19 Apr 2011 10:12:00 -0000 Message-ID: <005801cbfe7a$414b24e0$c3e16ea0$@muller@ics-cnrs.unistra.fr> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2011-04/txt/msg00300.txt.bz2 ARI UCASE rule lists 4 sources that have function starting with a capital letter. gnu-nat.c is a special case with functions starting with 'S_' that are recognized by exc_server, and should thus not be touched. The three other sources involve static functions for which I can see no real need of this uppercase function names. Is this patch OK or are there any valid reasons to use uppercase function like LM_NEXT? Pierre Muller GDB pascal language maintainer as ARI maintainer. 2011-04-19 Pierre Muller ARI fixes: Use only lowercase function name for static functions. * nto-tdep.c (LM_ADDR): Rename to... (lm_addr): New function name. (nto_relocate_section_addresses): Adapt to change above. * solib-sunos.c (LM_ADDR): Rename to... (lm_addr): New function name. (LM_NEXT): Rename to... (lm_next): New function name. (sunos_current_sos, sunos_relocate_section_addresses): Adapt to function name changes above. * solib-svr4.c (LM_ADDR_FROM_LINK_MAP): Rename to... (lm_addr_from_link_map): New function name. (HAS_LM_DYNAMIC_FROM_LINK_MAP): Rename to... (has_lm_dynamic_from_link_map): New function name. (LM_DYNAMIC_FROM_LINK_MAP): Rename to... (lm_dynamic_from_link_map): New function name. (LM_ADDR_CHECK): Rename to... (lm_addr_check): New function name. (LM_NEXT): Rename to... (lm_next): New function name. (LM_PREV): Rename to... (lm_prev): New function name. (LM_NAME): Rename to... (lm_name): New function name. (IGNORE_FIRST_LINK_MAP_ENTRY): Rename to... (ignore_first_link_map_entry): New function name. (svr4_keep_data_in_core): Adapt to function name changes above. (svr4_current_sos): Likewise. (enable_break): Likewise. (svr4_relocate_section_addresses): Likewise. Index: nto-tdep.c =================================================================== RCS file: /cvs/src/src/gdb/nto-tdep.c,v retrieving revision 1.40 diff -u -p -r1.40 nto-tdep.c --- nto-tdep.c 9 Mar 2011 15:01:36 -0000 1.40 +++ nto-tdep.c 19 Apr 2011 09:55:12 -0000 @@ -233,7 +233,7 @@ nto_parse_redirection (char *pargv[], co return argv; } -/* The struct lm_info, LM_ADDR, and nto_truncate_ptr are copied from +/* The struct lm_info, lm_addr, and nto_truncate_ptr are copied from solib-svr4.c to support nto_relocate_section_addresses which is different from the svr4 version. */ @@ -259,7 +259,7 @@ struct lm_info static CORE_ADDR -LM_ADDR (struct so_list *so) +lm_addr (struct so_list *so) { if (so->lm_info->l_addr == (CORE_ADDR)-1) { @@ -310,8 +310,8 @@ nto_relocate_section_addresses (struct s Elf_Internal_Phdr *phdr = find_load_phdr (sec->bfd); unsigned vaddr = phdr ? phdr->p_vaddr : 0; - sec->addr = nto_truncate_ptr (sec->addr + LM_ADDR (so) - vaddr); - sec->endaddr = nto_truncate_ptr (sec->endaddr + LM_ADDR (so) - vaddr); + sec->addr = nto_truncate_ptr (sec->addr + lm_addr (so) - vaddr); + sec->endaddr = nto_truncate_ptr (sec->endaddr + lm_addr (so) - vaddr); } /* This is cheating a bit because our linker code is in libc.so. If we Index: solib-sunos.c =================================================================== RCS file: /cvs/src/src/gdb/solib-sunos.c,v retrieving revision 1.43 diff -u -p -r1.43 solib-sunos.c --- solib-sunos.c 11 Jan 2011 21:53:24 -0000 1.43 +++ solib-sunos.c 19 Apr 2011 09:55:13 -0000 @@ -139,7 +139,7 @@ static CORE_ADDR flag_addr; /* link map access functions */ static CORE_ADDR -LM_ADDR (struct so_list *so) +lm_addr (struct so_list *so) { enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch); int lm_addr_offset = offsetof (struct link_map, lm_addr); @@ -150,7 +150,7 @@ LM_ADDR (struct so_list *so) } static CORE_ADDR -LM_NEXT (struct so_list *so) +lm_next (struct so_list *so) { enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch); int lm_next_offset = offsetof (struct link_map, lm_next); @@ -162,7 +162,7 @@ LM_NEXT (struct so_list *so) } static CORE_ADDR -LM_NAME (struct so_list *so) +lm_name (struct so_list *so) { enum bfd_endian byte_order = gdbarch_byte_order (target_gdbarch); int lm_name_offset = offsetof (struct link_map, lm_name); @@ -435,10 +435,10 @@ sunos_current_sos (void) read_memory (lm, new->lm_info->lm, sizeof (struct link_map)); - lm = LM_NEXT (new); + lm = lm_next (new); /* Extract this shared object's name. */ - target_read_string (LM_NAME (new), &buffer, + target_read_string (lm_name (new), &buffer, SO_NAME_MAX_PATH_SIZE - 1, &errcode); if (errcode != 0) warning (_("Can't read pathname for load map: %s."), @@ -825,8 +825,8 @@ static void sunos_relocate_section_addresses (struct so_list *so, struct target_section *sec) { - sec->addr += LM_ADDR (so); - sec->endaddr += LM_ADDR (so); + sec->addr += lm_addr (so); + sec->endaddr += lm_addr (so); } static struct target_so_ops sunos_so_ops; Index: solib-svr4.c =================================================================== RCS file: /cvs/src/src/gdb/solib-svr4.c,v retrieving revision 1.146 diff -u -p -r1.146 solib-svr4.c --- solib-svr4.c 17 Apr 2011 19:17:10 -0000 1.146 +++ solib-svr4.c 19 Apr 2011 10:04:05 -0000 @@ -143,7 +143,7 @@ svr4_same (struct so_list *gdb, struct s /* link map access functions. */ static CORE_ADDR -LM_ADDR_FROM_LINK_MAP (struct so_list *so) +lm_addr_from_link_map (struct so_list *so) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr; @@ -153,7 +153,7 @@ LM_ADDR_FROM_LINK_MAP (struct so_list *s } static int -HAS_LM_DYNAMIC_FROM_LINK_MAP (void) +has_lm_dynamic_from_link_map (void) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); @@ -161,7 +161,7 @@ HAS_LM_DYNAMIC_FROM_LINK_MAP (void) } static CORE_ADDR -LM_DYNAMIC_FROM_LINK_MAP (struct so_list *so) +lm_dynamic_from_link_map (struct so_list *so) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr; @@ -171,19 +171,19 @@ LM_DYNAMIC_FROM_LINK_MAP (struct so_list } static CORE_ADDR -LM_ADDR_CHECK (struct so_list *so, bfd *abfd) +lm_addr_check (struct so_list *so, bfd *abfd) { if (so->lm_info->l_addr == (CORE_ADDR)-1) { struct bfd_section *dyninfo_sect; CORE_ADDR l_addr, l_dynaddr, dynaddr; - l_addr = LM_ADDR_FROM_LINK_MAP (so); + l_addr = lm_addr_from_link_map (so); - if (! abfd || ! HAS_LM_DYNAMIC_FROM_LINK_MAP ()) + if (! abfd || ! has_lm_dynamic_from_link_map ()) goto set_addr; - l_dynaddr = LM_DYNAMIC_FROM_LINK_MAP (so); + l_dynaddr = lm_dynamic_from_link_map (so); dyninfo_sect = bfd_get_section_by_name (abfd, ".dynamic"); if (dyninfo_sect == NULL) @@ -273,7 +273,7 @@ LM_ADDR_CHECK (struct so_list *so, bfd * } static CORE_ADDR -LM_NEXT (struct so_list *so) +lm_next (struct so_list *so) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr; @@ -283,7 +283,7 @@ LM_NEXT (struct so_list *so) } static CORE_ADDR -LM_PREV (struct so_list *so) +lm_prev (struct so_list *so) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr; @@ -293,7 +293,7 @@ LM_PREV (struct so_list *so) } static CORE_ADDR -LM_NAME (struct so_list *so) +lm_name (struct so_list *so) { struct link_map_offsets *lmo = svr4_fetch_link_map_offsets (); struct type *ptr_type = builtin_type (target_gdbarch)->builtin_data_ptr; @@ -303,14 +303,14 @@ LM_NAME (struct so_list *so) } static int -IGNORE_FIRST_LINK_MAP_ENTRY (struct so_list *so) +ignore_first_link_map_entry (struct so_list *so) { /* Assume that everything is a library if the dynamic loader was loaded late by a static executable. */ if (exec_bfd && bfd_get_section_by_name (exec_bfd, ".dynamic") == NULL) return 0; - return LM_PREV (so) == 0; + return lm_prev (so) == 0; } /* Per pspace SVR4 specific data. */ @@ -977,7 +977,7 @@ svr4_keep_data_in_core (CORE_ADDR vaddr, new->lm_info->lm = xzalloc (lmo->link_map_size); make_cleanup (xfree, new->lm_info->lm); read_memory (ldsomap, new->lm_info->lm, lmo->link_map_size); - lm_name = LM_NAME (new); + lm_name = lm_name (new); do_cleanups (old_chain); return (lm_name >= vaddr && lm_name < vaddr + size); @@ -1158,9 +1158,9 @@ svr4_current_sos (void) read_memory (lm, new->lm_info->lm, lmo->link_map_size); - next_lm = LM_NEXT (new); + next_lm = lm_next (new); - if (LM_PREV (new) != prev_lm) + if (lm_prev (new) != prev_lm) { warning (_("Corrupted shared library list")); free_so (new); @@ -1172,7 +1172,7 @@ svr4_current_sos (void) SVR4, it has no name. For others (Solaris 2.3 for example), it does have a name, so we can no longer use a missing name to decide when to ignore it. */ - else if (IGNORE_FIRST_LINK_MAP_ENTRY (new) && ldsomap == 0) + else if (ignore_first_link_map_entry (new) && ldsomap == 0) { info->main_lm_addr = new->lm_info->lm_addr; free_so (new); @@ -1183,7 +1183,7 @@ svr4_current_sos (void) char *buffer; /* Extract this shared object's name. */ - target_read_string (LM_NAME (new), &buffer, + target_read_string (lm_name (new), &buffer, SO_NAME_MAX_PATH_SIZE - 1, &errcode); if (errcode != 0) warning (_("Can't read pathname for load map: %s."), @@ -1484,7 +1484,7 @@ enable_break (struct svr4_info *info, in { load_addr_found = 1; loader_found_in_list = 1; - load_addr = LM_ADDR_CHECK (so, tmp_bfd); + load_addr = lm_addr_check (so, tmp_bfd); break; } so = so->next; @@ -2270,9 +2270,9 @@ static void svr4_relocate_section_addresses (struct so_list *so, struct target_section *sec) { - sec->addr = svr4_truncate_ptr (sec->addr + LM_ADDR_CHECK (so, + sec->addr = svr4_truncate_ptr (sec->addr + lm_addr_check (so, sec->bfd)); - sec->endaddr = svr4_truncate_ptr (sec->endaddr + LM_ADDR_CHECK (so, + sec->endaddr = svr4_truncate_ptr (sec->endaddr + lm_addr_check (so, sec->bfd)); }