[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Libcdio-devel] a few small windows fixes
From: |
Ozkan Sezer |
Subject: |
Re: [Libcdio-devel] a few small windows fixes |
Date: |
Sat, 25 Nov 2017 15:55:16 +0300 |
On 11/25/17, Ozkan Sezer <address@hidden> wrote:
> See the attached three patches:
>
>
> #1: fix mixed declarations and code in win32_ioctl.c:
>
> lib/driver/MSWindows/win32_ioctl.c: In function
> 'audio_play_msf_win32ioctl':
> lib/driver/MSWindows/win32_ioctl.c:193:3: warning: ISO C90 forbids
> mixed declarations and code
>
>
> #2: fix 'strndup shadows a built-in function' warnings:
>
> if strndup() is not available, the build log is plagued with the
> following warnings:
> lib/driver/cdio_private.h:46:21: warning: declaration of 'strndup'
> shadows a built-in function
>
> So, rename the replacement inline strndup to libcdio_strdup, and
> define strndup as libcdio_strndup.
>
>
> #3: do not use sleep() for windows.
>
> _sleep() has long been deprecated and no longer prototyped in stdlib.h,
> despite the fact that msvcrt.dll still exports it. therefore, undefine
> HAVE_SLEEP when targeting windows, use Sleep win32 api function instead.
The list archive doesn't contain the attached patches. Some ridiculous
filter ate them?? Re-attaching them after renaming *.patch to *.txt,
maybe they arrive this time..
--
O.S.
0001-fix-mixed-declarations-and-code-in-win32_ioctl.c.txt
Description: Text document
0002-fix-strndup-shadows-a-built-in-function-warnings.txt
Description: Text document
0003-do-not-use-sleep-for-windows.txt
Description: Text document