mirror of
https://github.com/espressif/binutils-gdb.git
synced 2025-06-27 14:39:09 +08:00
gcc lint
This commit is contained in:
@ -1,5 +1,25 @@
|
|||||||
Thu Jan 19 14:49:47 1995 Ken Raeburn <raeburn@cujo.cygnus.com>
|
Thu Jan 19 14:49:47 1995 Ken Raeburn <raeburn@cujo.cygnus.com>
|
||||||
|
|
||||||
|
* config/tc-alpha.c (alpha_ip): Initialize local variables to keep
|
||||||
|
gcc quiet.
|
||||||
|
(gpdisp_hi16_howto): Don't use "const" with reloc_howto_type.
|
||||||
|
(in_range_signed): Add parens around subtraction inside shift.
|
||||||
|
* gasp.c (do_data): Initialize local variable "opname".
|
||||||
|
(istrue): Provide a default case to set "res" and keep "gcc -Wall"
|
||||||
|
quiet.
|
||||||
|
* write.c (write_contents): Deleted unused variable.
|
||||||
|
(print_symbol_value_1): Declare.
|
||||||
|
* hash.c (hash_ask): Delete disabled non-strcmp version of the
|
||||||
|
code, and automatic variables used only in those sections.
|
||||||
|
* write.c (chain_frchains_together_1): Only define local variable
|
||||||
|
"prev_fix" if BFD_ASSEMBLER.
|
||||||
|
* flonum-konst.c (dummy1): Return void.
|
||||||
|
* config/tc-vax.c (md_assemble): Remove two comparisons of
|
||||||
|
unsigned numbers versus zero.
|
||||||
|
|
||||||
|
* config/tc-alpha.c, config/alpha-opcode.h: Revert 2 June changes.
|
||||||
|
Turns out we never got the assignment done after all.
|
||||||
|
|
||||||
Cleanup of VAX and VMS code, from Pat Rankin:
|
Cleanup of VAX and VMS code, from Pat Rankin:
|
||||||
* config/obj-vms.c: Changed exported function names to lower case.
|
* config/obj-vms.c: Changed exported function names to lower case.
|
||||||
(Changed call sites in write.c.) Declare VMS system function
|
(Changed call sites in write.c.) Declare VMS system function
|
||||||
|
@ -1494,7 +1494,7 @@ do_data (idx, in, size)
|
|||||||
int size;
|
int size;
|
||||||
{
|
{
|
||||||
int opsize = 4;
|
int opsize = 4;
|
||||||
char *opname;
|
char *opname = ".yikes!";
|
||||||
sb acc;
|
sb acc;
|
||||||
sb_new (&acc);
|
sb_new (&acc);
|
||||||
|
|
||||||
@ -2351,6 +2351,9 @@ istrue (idx, in)
|
|||||||
idx = exp_get_abs ("Conditional operator must have absolute operands.\n", idx, in, &valb);
|
idx = exp_get_abs ("Conditional operator must have absolute operands.\n", idx, in, &valb);
|
||||||
switch (cond)
|
switch (cond)
|
||||||
{
|
{
|
||||||
|
default:
|
||||||
|
res = 42;
|
||||||
|
break;
|
||||||
case EQ:
|
case EQ:
|
||||||
res = vala == valb;
|
res = vala == valb;
|
||||||
break;
|
break;
|
||||||
|
29
gas/hash.c
29
gas/hash.c
@ -678,9 +678,7 @@ hash_ask (handle, string, access)
|
|||||||
const char *string;
|
const char *string;
|
||||||
int access; /* access type */
|
int access; /* access type */
|
||||||
{
|
{
|
||||||
const char *string1; /* JF avoid strcmp calls */
|
|
||||||
const char *s;
|
const char *s;
|
||||||
int c;
|
|
||||||
struct hash_entry *slot;
|
struct hash_entry *slot;
|
||||||
int collision; /* count collisions */
|
int collision; /* count collisions */
|
||||||
|
|
||||||
@ -692,22 +690,8 @@ hash_ask (handle, string, access)
|
|||||||
hash_found = FALSE;
|
hash_found = FALSE;
|
||||||
while (((s = slot->hash_string) != NULL) && s != DELETED)
|
while (((s = slot->hash_string) != NULL) && s != DELETED)
|
||||||
{
|
{
|
||||||
#if 1
|
|
||||||
if (string == s || !strcmp (string, s))
|
if (string == s || !strcmp (string, s))
|
||||||
hash_found = TRUE;
|
hash_found = TRUE;
|
||||||
#else
|
|
||||||
for (string1 = string;;)
|
|
||||||
{
|
|
||||||
if ((c = *s++) == 0)
|
|
||||||
{
|
|
||||||
if (!*string1)
|
|
||||||
hash_found = TRUE;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
if (*string1++ != c)
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
#endif
|
|
||||||
if (hash_found)
|
if (hash_found)
|
||||||
break;
|
break;
|
||||||
collision++;
|
collision++;
|
||||||
@ -726,21 +710,8 @@ hash_ask (handle, string, access)
|
|||||||
slot = handle->hash_where;/* now look again */
|
slot = handle->hash_where;/* now look again */
|
||||||
while (((s = slot->hash_string) != NULL) && s != DELETED)
|
while (((s = slot->hash_string) != NULL) && s != DELETED)
|
||||||
{
|
{
|
||||||
#if 0
|
|
||||||
for (string1 = string; *s; string1++, s++)
|
|
||||||
{
|
|
||||||
if (*string1 != *s)
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
if (*s == *string1)
|
|
||||||
{
|
|
||||||
hash_found = TRUE;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
#else
|
|
||||||
if (string == s || !strcmp (string, s))
|
if (string == s || !strcmp (string, s))
|
||||||
hash_found = TRUE;
|
hash_found = TRUE;
|
||||||
#endif
|
|
||||||
collision++;
|
collision++;
|
||||||
slot++;
|
slot++;
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user