John Marriott wrote:
> On 23-July-2022 14:05, Bram Moolenaar wrote:
> > Patch 9.0.0058
> > Problem: Win32: cannot test low level events.
> > Solution: Add "sendevent" to test_gui_event(). (Yegappan Lakshmanan,
> > closes #10679)
> > Files: runtime/doc/testing.txt, src/errors.h, src/gui_w32.c,
> > src/proto/gui_w32.pro, src/testing.c, src/testdir/test_gui.vim
> >
> >
> After this patch, mingw64 (gcc 12.1.0) throws this warning:
> <snip>
> gcc -c -I. -Iproto -DWIN32 -DWINVER=0x0603 -D_WIN32_WINNT=0x0603
> -DHAVE_PATHDEF -DFEAT_NORMAL -DHAVE_STDINT_H -D__USE_MINGW_ANSI_STDIO
> -pipe -march=native -Wall -O3 -fomit-frame-pointer -freg-struct-return
> -fpie -fPIE -DFEAT_GUI_MSWIN -DFEAT_CLIPBOARD gui_w32.c -o
> gobjnative/gui_w32.o
> gui_w32.c:8552:35: warning: pointer targets in passing argument 2 of
> 'dict_get_string' differ in signedness [-Wpointer-sign]
> 8552 | event = dict_get_string(args, "event", TRUE);
> | ^~~~~~~
> | |
> | char *
> In file included from proto.h:75,
> from vim.h:2263,
> from gui_w32.c:26:
> proto/dict.pro:32:44: note: expected 'char_u *' {aka 'unsigned char *'}
> but argument is of type 'char *'
> 32 | char_u *dict_get_string(dict_T *d, char_u *key, int save);
> | ~~~~~~~~^~~
> gui_w32.c:8562:44: warning: pointer targets in passing argument 2 of
> 'dict_get_number_def' differ in signedness [-Wpointer-sign]
> 8562 | vkCode = dict_get_number_def(args, "keycode", 0);
> | ^~~~~~~~~
> | |
> | char *
> proto/dict.pro:34:52: note: expected 'char_u *' {aka 'unsigned char *'}
> but argument is of type 'char *'
> 34 | varnumber_T dict_get_number_def(dict_T *d, char_u *key, int def);
> | ~~~~~~~~^~~
> </snip>
>
> The attached patch tries to fix it.
Thanks!
It looks like nearly all of these calls use a string constant, I'll make
the argument "char *" to avoid a lot of type casts.
--
Contrary to popular belief, Unix is user friendly.
It just happens to be selective about who it makes friends with.
-- Dave Parnas
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/vim_dev/20220723082414.E901D1C0909%40moolenaar.net.
