Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
Fix build failures with -DDEBUG.
        [BZ #19443]
        * crypt/crypt_util.c [DEBUG] (_ufc_prbits): Correct format string.
        [DEBUG] (_ufc_set_bits): Declare used.
        * iconv/gconv_dl.c [DEBUG]: Add a missing include directive.
        [DEBUG] (print_all): Declare used.
        * resolv/res_send.c [DEBUG] (__libc_res_nsend): Explicitly convert
        operands of the ternary ?: expression to target type.
        * stdlib/rshift.c [DEBUG] (mpn_rshift): Use assert() instead of
        calling the undeclared abort.
        * time/mktime.c [DEBUG] (DEBUG): Rename to DEBUG_MKTIME.
  • Loading branch information
Martin Sebor committed Jan 15, 2016
1 parent ad37480 commit f2b3078
Show file tree
Hide file tree
Showing 6 changed files with 30 additions and 15 deletions.
13 changes: 13 additions & 0 deletions ChangeLog
@@ -1,3 +1,16 @@
2016-01-15 Martin Sebor <msebor@redhat.com>

[BZ #19443]
* crypt/crypt_util.c [DEBUG] (_ufc_prbits): Correct format string.
[DEBUG] (_ufc_set_bits): Declare used.
* iconv/gconv_dl.c [DEBUG]: Add a missing include directive.
[DEBUG] (print_all): Declare used.
* resolv/res_send.c [DEBUG] (__libc_res_nsend): Explicitly convert
operands of the ternary ?: expression to target type.
* stdlib/rshift.c [DEBUG] (mpn_rshift): Use assert() instead of
calling the undeclared abort.
* time/mktime.c [DEBUG] (DEBUG): Rename to DEBUG_MKTIME.

2016-01-15 Martin Sebor <msebor@redhat.com>

[BZ #18755]
Expand Down
4 changes: 2 additions & 2 deletions crypt/crypt_util.c
Expand Up @@ -271,12 +271,12 @@ _ufc_prbits (ufc_long *a, int n)
t=8*i+j;
tmp|=(a[t/24] & BITMASK[t % 24])?bytemask[j]:0;
}
(void)printf("%02x ",tmp);
(void)printf("%02lx ", tmp);
}
printf(" ");
}

static void
static void __attribute__ ((unused))
_ufc_set_bits (ufc_long v, ufc_long *b)
{
ufc_long i;
Expand Down
5 changes: 4 additions & 1 deletion iconv/gconv_dl.c
Expand Up @@ -219,6 +219,9 @@ libc_freeres_fn (free_mem)


#ifdef DEBUG

#include <stdio.h>

static void
do_print (const void *nodep, VISIT value, int level)
{
Expand All @@ -231,7 +234,7 @@ do_print (const void *nodep, VISIT value, int level)
obj->name, obj->counter);
}

static void
static void __attribute__ ((used))
print_all (void)
{
__twalk (loaded, do_print);
Expand Down
4 changes: 2 additions & 2 deletions resolv/res_send.c
Expand Up @@ -499,8 +499,8 @@ __libc_res_nsend(res_state statp, const u_char *buf, int buflen,
(stdout, ";; Querying server (# %d) address = %s\n",
ns + 1, inet_ntop(nsap->sa_family,
(nsap->sa_family == AF_INET6
? &((struct sockaddr_in6 *) nsap)->sin6_addr
: &((struct sockaddr_in *) nsap)->sin_addr),
? (void *) &((struct sockaddr_in6 *) nsap)->sin6_addr
: (void *) &((struct sockaddr_in *) nsap)->sin_addr),
tmpbuf, sizeof (tmpbuf))));

if (__glibc_unlikely (v_circuit)) {
Expand Down
7 changes: 3 additions & 4 deletions stdlib/rshift.c
Expand Up @@ -21,6 +21,8 @@ along with the GNU MP Library; see the file COPYING.LIB. If not, see
#include <gmp.h>
#include "gmp-impl.h"

#include <assert.h>

/* Shift U (pointed to by UP and USIZE limbs long) CNT bits to the right
and store the USIZE least significant limbs of the result at WP.
The bits shifted out to the right are returned.
Expand All @@ -40,10 +42,7 @@ mpn_rshift (register mp_ptr wp,
register mp_size_t i;
mp_limb_t retval;

#ifdef DEBUG
if (usize == 0 || cnt == 0)
abort ();
#endif
assert (usize != 0 && cnt != 0);

sh_1 = cnt;

Expand Down
12 changes: 6 additions & 6 deletions time/mktime.c
Expand Up @@ -19,7 +19,7 @@

/* Define this to have a standalone program to test this implementation of
mktime. */
/* #define DEBUG 1 */
/* #define DEBUG_MKTIME 1 */

#ifndef _LIBC
# include <config.h>
Expand All @@ -38,13 +38,13 @@

#include <string.h> /* For the real memcpy prototype. */

#if defined DEBUG && DEBUG
#if defined DEBUG_MKTIME && DEBUG_MKTIME
# include <stdio.h>
# include <stdlib.h>
/* Make it work even if the system's libc has its own mktime routine. */
# undef mktime
# define mktime my_mktime
#endif /* DEBUG */
#endif /* DEBUG_MKTIME */

/* Some of the code in this file assumes that signed integer overflow
silently wraps around. This assumption can't easily be programmed
Expand Down Expand Up @@ -600,7 +600,7 @@ libc_hidden_def (mktime)
libc_hidden_weak (timelocal)
#endif

#if defined DEBUG && DEBUG
#if defined DEBUG_MKTIME && DEBUG_MKTIME

static int
not_equal_tm (const struct tm *a, const struct tm *b)
Expand Down Expand Up @@ -732,10 +732,10 @@ main (int argc, char **argv)
return status;
}

#endif /* DEBUG */
#endif /* DEBUG_MKTIME */

/*
Local Variables:
compile-command: "gcc -DDEBUG -I. -Wall -W -O2 -g mktime.c -o mktime"
compile-command: "gcc -DDEBUG_MKTIME -I. -Wall -W -O2 -g mktime.c -o mktime"
End:
*/

0 comments on commit f2b3078

Please sign in to comment.