diff options
author | Martin Storsjö <martin@martin.st> | 2023-12-18 10:48:28 +0200 |
---|---|---|
committer | Martin Storsjö <martin@martin.st> | 2023-12-19 00:07:26 +0200 |
commit | f80b2ce9fa3235ce0e627d54df25e3f6cc929c05 (patch) | |
tree | 8e46e6c4060091630e8aa446ed2885cbad02f06b /tests/ref/fate/dds-dx10-bc2 | |
parent | b391fdbf1aedbf9a77ef3079607f5d54f9d78309 (diff) | |
download | ffmpeg-f80b2ce9fa3235ce0e627d54df25e3f6cc929c05.tar.gz |
gdigrab: Fix hwnd parameter issues
Converting from an integer to HWND (which is a pointer) requires
an explicit cast, otherwise Clang errors out like this:
src/libavdevice/gdigrab.c:280:14: error: incompatible integer to pointer conversion assigning to 'HWND' (aka 'struct HWND__ *') from 'long' [-Wint-conversion]
280 | hwnd = strtol(name, &p, 0);
| ^ ~~~~~~~~~~~~~~~~~~~
(With GCC and MSVC, this was a mere warning, but with recent Clang,
this is an error.)
After adding a cast, all compilers also warn something like this:
src/libavdevice/gdigrab.c:280:16: warning: cast to 'HWND' (aka 'struct HWND__ *') from smaller integer type 'long' [-Wint-to-pointer-cast]
280 | hwnd = (HWND) strtol(name, &p, 0);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
On Windows, long types are 32 bit, so to get a usable pointer, we
need to use long long. And interpret it as unsigned long long
while at it - i.e. using strtoull.
Finally, right above it, the code triggered the following warning:
src/libavdevice/gdigrab.c:278:15: warning: mixing declarations and code is incompatible with standards before C99 [-Wdeclaration-after-statement]
278 | char *p;
| ^
Signed-off-by: Martin Storsjö <martin@martin.st>
Diffstat (limited to 'tests/ref/fate/dds-dx10-bc2')
0 files changed, 0 insertions, 0 deletions