[Git][NTPsec/ntpsec][master] Revert "Attempt to suppress some compiler warnings."

Eric S. Raymond gitlab at mg.gitlab.com
Thu Sep 7 19:19:39 UTC 2017


Eric S. Raymond pushed to branch master at NTPsec / ntpsec


Commits:
343448b7 by Eric S. Raymond at 2017-09-07T15:18:38-04:00
Revert "Attempt to suppress some compiler warnings."

It doesn't do any good, and it breaks the build on CentOS.

- - - - -


2 changed files:

- include/recvbuff.h
- libntp/clocktime.c


Changes:

=====================================
include/recvbuff.h
=====================================
--- a/include/recvbuff.h
+++ b/include/recvbuff.h
@@ -51,7 +51,7 @@ struct recvbuf {
 	union {
 		struct pkt	X_recv_pkt;
 		uint8_t		X_recv_buffer[RX_BUFF_SIZE];
-	} recv_space __attribute__ ((aligned));
+	} recv_space;
 #define	recv_pkt		recv_space.X_recv_pkt
 #define	recv_buffer		recv_space.X_recv_buffer
 	int used;		/* reference count */


=====================================
libntp/clocktime.c
=====================================
--- a/libntp/clocktime.c
+++ b/libntp/clocktime.c
@@ -91,8 +91,6 @@ clocktime(
 	 */
 	if (year > 1970) {
 	    *yearstart = year_to_ntp(year);
-	    /* adding a signed int to an unsigned seems to confuse gcc */
-#pragma GCC diagnostic ignored "-Wsign-conversion"
 	    *ts_ui = *yearstart + tmp;
 	    return true;
 	}



View it on GitLab: https://gitlab.com/NTPsec/ntpsec/commit/343448b71e7c9df6a0266211727ae894be8f15f5

---
View it on GitLab: https://gitlab.com/NTPsec/ntpsec/commit/343448b71e7c9df6a0266211727ae894be8f15f5
You're receiving this email because of your account on gitlab.com.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ntpsec.org/pipermail/vc/attachments/20170907/96d23533/attachment.html>


More information about the vc mailing list