From: Eli Zaretskii <eliz@gnu.org>
To: Andy Moreton <andrewjmoreton@gmail.com>
Cc: emacs-devel@gnu.org
Subject: Re: Patch for warnings in 64bit Windows builds
Date: Fri, 31 Jul 2015 09:55:43 +0300 [thread overview]
Message-ID: <83fv44x1cg.fsf@gnu.org> (raw)
In-Reply-To: <86y4hxl5cw.fsf@gmail.com>
> From: Andy Moreton <andrewjmoreton@gmail.com>
> Date: Thu, 30 Jul 2015 22:09:51 +0100
>
> I've noticed some other warnings in the 64bit mingw64 builds requiring
> a similar fix:
>
> ../../src/w32.c: In function 'maybe_load_unicows_dll':
> ../../src/w32.c:9174:25: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types]
> pMultiByteToWideChar = GetProcAddress (ret, "MultiByteToWideChar");
> ^
> ../../src/w32.c:9175:25: warning: assignment from incompatible pointer type [-Wincompatible-pointer-types]
> pWideCharToMultiByte = GetProcAddress (ret, "WideCharToMultiByte");
> ^
>
> There is also a warning about socket handling:
>
> ../../src/w32.c: In function 'sys_socket':
> ../../src/w32.c:7336:14: warning: overflow in implicit constant conversion [-Woverflow]
> return INVALID_SOCKET;
> ^
> c:/msys64/mingw64/x86_64-w64-mingw32/include/psdk_inc/_socket_types.h
> contains:
>
> #if 1
> typedef UINT_PTR SOCKET;
> #else
> typedef INT_PTR SOCKET;
> #endif
>
> #define INVALID_SOCKET (SOCKET)(~0)
>
> Thus it is corect to warn about a conversion from unsigned 64bit to a
> signed 32bit value.
Please propose changes to eliminate those. (I can propose them
myself, but I cannot test them.)
> >> I don't have copyright paperwork on file, so please let me know if that
> >> is required.
> >
> > Not needed, but you are encouraged to start the paperwork rolling, so
> > it will be ready for your next contribution.
>
> Where do I find the appropriate forms ?
One place is in Gnulib's Git repository, in the doc/Copyright/
directory. You want the file request-assign.future. There are some
explanations in
https://www.gnu.org/prep/maintain/html_node/Copyright-Papers.html, but
the form is pretty much self-explanatory.
Thanks.
prev parent reply other threads:[~2015-07-31 6:55 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-28 11:57 Patch for warnings in 64bit Windows builds Andy Moreton
2015-07-28 13:40 ` Eli Zaretskii
2015-07-30 21:09 ` Andy Moreton
2015-07-31 6:55 ` Eli Zaretskii [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=83fv44x1cg.fsf@gnu.org \
--to=eliz@gnu.org \
--cc=andrewjmoreton@gmail.com \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.