GNU bug report logs - #38040
Compiler warnings on win32 build

Previous Next

Package: emacs;

Reported by: Cecilio Pardo <cpardo <at> imayhem.com>

Date: Sun, 3 Nov 2019 00:37:02 UTC

Severity: wishlist

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 38040 in the body.
You can then email your comments to 38040 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#38040; Package emacs. (Sun, 03 Nov 2019 00:37:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Cecilio Pardo <cpardo <at> imayhem.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Sun, 03 Nov 2019 00:37:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Cecilio Pardo <cpardo <at> imayhem.com>
To: bug-gnu-emacs <at> gnu.org
Subject: Compiler warnings on win32 build
Date: Sun, 03 Nov 2019 01:36:41 +0100
Severity: wishlist
Tags: patch

Building with mingw64, there are a couple of warnings about wrong data
types that can be avoided:


diff --git a/nt/addpm.c b/nt/addpm.c
index a8bcd4a5c7..ff169f7d87 100644
--- a/nt/addpm.c
+++ b/nt/addpm.c
@@ -56,13 +56,13 @@ #define COBJMACROS 1
 #include "../src/epaths.h"
 #endif
 
-HDDEDATA CALLBACK DdeCallback (UINT, UINT, HCONV, HSZ, HSZ, HDDEDATA, DWORD,
-			       DWORD);
+HDDEDATA CALLBACK DdeCallback (UINT, UINT, HCONV, HSZ, HSZ, HDDEDATA, ULONG_PTR,
+			       ULONG_PTR);
 
 HDDEDATA CALLBACK
 DdeCallback (UINT uType, UINT uFmt, HCONV hconv,
 	     HSZ hsz1, HSZ hsz2, HDDEDATA hdata,
-	     DWORD dwData1, DWORD dwData2)
+	     ULONG_PTR dwData1, ULONG_PTR dwData2)
 {
   return ((HDDEDATA) NULL);
 }
diff --git a/nt/ddeclient.c b/nt/ddeclient.c
index e441700397..a63be13f75 100644
--- a/nt/ddeclient.c
+++ b/nt/ddeclient.c
@@ -21,13 +21,13 @@
 #include <stdlib.h>
 #include <stdio.h>
 
-HDDEDATA CALLBACK DdeCallback (UINT, UINT, HCONV, HSZ, HSZ, HDDEDATA, DWORD,
-			       DWORD);
+HDDEDATA CALLBACK DdeCallback (UINT, UINT, HCONV, HSZ, HSZ, HDDEDATA, ULONG_PTR,
+			       ULONG_PTR);
 
 HDDEDATA CALLBACK
 DdeCallback (UINT uType, UINT uFmt, HCONV hconv,
 	     HSZ hsz1, HSZ hsz2, HDDEDATA hdata,
-	     DWORD dwData1, DWORD dwData2)
+	     ULONG_PTR dwData1, ULONG_PTR dwData2)
 {
   return ((HDDEDATA) NULL);
 }








Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#38040; Package emacs. (Sat, 09 Nov 2019 09:56:02 GMT) Full text and rfc822 format available.

Message #8 received at 38040 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Cecilio Pardo <cpardo <at> imayhem.com>
Cc: 38040 <at> debbugs.gnu.org
Subject: Re: bug#38040: Compiler warnings on win32 build
Date: Sat, 09 Nov 2019 11:55:05 +0200
> From: Cecilio Pardo <cpardo <at> imayhem.com>
> Date: Sun, 03 Nov 2019 01:36:41 +0100
> 
> Building with mingw64, there are a couple of warnings about wrong data
> types that can be avoided:

Thanks, I installed a slightly different change, because these types
are declared DWORD in mingw.org's MinGW headers.  Please see that the
warnings are gone in your build.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#38040; Package emacs. (Sun, 10 Nov 2019 00:05:02 GMT) Full text and rfc822 format available.

Message #11 received at 38040 <at> debbugs.gnu.org (full text, mbox):

From: Cecilio Pardo <cpardo <at> imayhem.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 38040 <at> debbugs.gnu.org, Cecilio Pardo <cpardo <at> imayhem.com>
Subject: Re: bug#38040: Compiler warnings on win32 build
Date: Sun, 10 Nov 2019 01:03:51 +0100
>> Building with mingw64, there are a couple of warnings about wrong data
>> types that can be avoided:
>
> Thanks, I installed a slightly different change, because these types
> are declared DWORD in mingw.org's MinGW headers.  Please see that the
> warnings are gone in your build.

They are gone. Thanks for fixing this.

-- 
Cecilio Pardo




Reply sent to Eli Zaretskii <eliz <at> gnu.org>:
You have taken responsibility. (Thu, 14 Nov 2019 09:19:02 GMT) Full text and rfc822 format available.

Notification sent to Cecilio Pardo <cpardo <at> imayhem.com>:
bug acknowledged by developer. (Thu, 14 Nov 2019 09:19:03 GMT) Full text and rfc822 format available.

Message #16 received at 38040-done <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Cecilio Pardo <cpardo <at> imayhem.com>
Cc: 38040-done <at> debbugs.gnu.org
Subject: Re: bug#38040: Compiler warnings on win32 build
Date: Thu, 14 Nov 2019 11:18:32 +0200
> From: Cecilio Pardo <cpardo <at> imayhem.com>
> Cc: Cecilio Pardo <cpardo <at> imayhem.com>,  38040 <at> debbugs.gnu.org
> Date: Sun, 10 Nov 2019 01:03:51 +0100
> 
> 
> >> Building with mingw64, there are a couple of warnings about wrong data
> >> types that can be avoided:
> >
> > Thanks, I installed a slightly different change, because these types
> > are declared DWORD in mingw.org's MinGW headers.  Please see that the
> > warnings are gone in your build.
> 
> They are gone. Thanks for fixing this.

Thanks, closing.




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 12 Dec 2019 12:24:09 GMT) Full text and rfc822 format available.

This bug report was last modified 5 years and 189 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.