commit/XEmacs: kehoea: Quieten some compiler warnings on 64 bit Linux, clang-3.8.

1 new commit in XEmacs: https://bitbucket.org/xemacs/xemacs/commits/52af36c1d478/ Changeset: 52af36c1d478 User: kehoea Date: 2017-11-07 17:59:18+00:00 Summary: Quieten some compiler warnings on 64 bit Linux, clang-3.8. src/ChangeLog addition: 2017-11-07 Aidan Kehoe <kehoea(a)parhasard.net&gt; Quieten some compiler warnings: * data.c (uint32_t_to_lisp): Add a check for sizeof (ITEM), which quietens the tautological-constant-compare warning on clang. * linuxplay.c (linux_play_data_or_file): Declare a ssize_t variable to take the result of read(). There are a lot of issues with the types used for bytecounts for the sound code, but I'm not going to fix them all today. * tls.c: Wrap a couple of declarations of Lisp variables in #ifdef WITH_TLS. Affected #: 4 files diff -r 6c493b7790729ec2b805a5967c9374258998613d -r 52af36c1d4781c2bce577f240c75f9d67cd5c2f5 src/ChangeLog --- a/src/ChangeLog +++ b/src/ChangeLog ＠＠ -1,3 +1,16 ＠＠ +2017-11-07 Aidan Kehoe <kehoea(a)parhasard.net&gt; + + Quieten some compiler warnings: + * data.c (uint32_t_to_lisp): + Add a check for sizeof (ITEM), which quietens the + tautological-constant-compare warning on clang. + * linuxplay.c (linux_play_data_or_file): + Declare a ssize_t variable to take the result of read(). There are + a lot of issues with the types used for bytecounts for the sound + code, but I'm not going to fix them all today. + * tls.c: + Wrap a couple of declarations of Lisp variables in #ifdef WITH_TLS. + 2017-11-07 Aidan Kehoe <kehoea(a)parhasard.net&gt; valid_ichar_p() needs an EMACS_INT argument, not an Ichar diff -r 6c493b7790729ec2b805a5967c9374258998613d -r 52af36c1d4781c2bce577f240c75f9d67cd5c2f5 src/data.c --- a/src/data.c +++ b/src/data.c ＠＠ -1153,7 +1153,8 ＠＠ Lisp_Object uint32_t_to_lisp (UINT_32_BIT item) { - if (item <= MOST_POSITIVE_FIXNUM_UNSIGNED) /* Fits in a positive fixnum? */ + if (sizeof (item) < sizeof (EMACS_INT) /* Fits in a positive fixnum? */ + || item <= MOST_POSITIVE_FIXNUM_UNSIGNED) { return make_fixnum (item); } diff -r 6c493b7790729ec2b805a5967c9374258998613d -r 52af36c1d4781c2bce577f240c75f9d67cd5c2f5 src/linuxplay.c --- a/src/linuxplay.c +++ b/src/linuxplay.c ＠＠ -338,8 +338,14 ＠＠ sound_warn(buf); goto END_OF_PLAY; } } if (fd >= 0) { - if ((rrtn = read(fd,sndbuf,SNDBUFSZ)) < 0) { - sound_perror("read"); goto END_OF_PLAY; } } + ssize_t gelesen; + if ((gelesen = read(fd,sndbuf,SNDBUFSZ)) < 0) + { + sound_perror("read"); + goto END_OF_PLAY; + } + rrtn = (size_t) gelesen; + } else break; } while (rrtn > 0); diff -r 6c493b7790729ec2b805a5967c9374258998613d -r 52af36c1d4781c2bce577f240c75f9d67cd5c2f5 src/tls.c --- a/src/tls.c +++ b/src/tls.c ＠＠ -28,8 +28,10 ＠＠ #include <netinet/in.h> #include <netinet/tcp.h> +#ifdef WITH_TLS static Lisp_Object prompt; static Lisp_Object Qread_passwd; +#endif Lisp_Object Qtls_error; #ifdef HAVE_NSS Repository URL: https://bitbucket.org/xemacs/xemacs/ -- This is a commit notification from bitbucket.org. You are receiving this because you have the service enabled, addressing the recipient of this email.