[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Libcdio-devel] [Pushed PATCH] Fix MinGW inlining failed warnings
From: |
Pete Batard |
Subject: |
[Libcdio-devel] [Pushed PATCH] Fix MinGW inlining failed warnings |
Date: |
Wed, 6 Jun 2018 16:43:42 +0100 |
User-agent: |
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
Hi,
I took the liberty of pushing a commit that removes inlining for 2
specific Win32 static calls in driver/image/bincue.c and driver/image/nrg.c.
Considering that recent compilers are usually smart enough about
inlining what can be inlined, I don't think this removal should harm us any.
This is to address the following warnings when compiling with MinGW32:
image/bincue.c: In function 'cdio_get_devices_bincue':
image/bincue.c:992:20: warning: inlining failed in call to
'Win32Glob.constprop': call is unlikely and code size would grow [-Winline]
static inline void Win32Glob(const char* pattern, const char*
szCurPath, char ***drives, unsigned int *num_files)
^~~~~~~~~
image/bincue.c:1016:7: note: called from here
Win32Glob(pattern, szPath, drives, num_files);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
image/nrg.c: In function 'cdio_get_devices_nrg':
image/nrg.c:1128:20: warning: inlining failed in call to
'Win32Glob.constprop': call is unlikely and code size would grow [-Winline]
static inline void Win32Glob(const char* pattern, const char*
szCurPath, char ***drives, unsigned int *num_files)
^~~~~~~~~
image/nrg.c:1152:7: note: called from here
Win32Glob(pattern, szPath, drives, num_files);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
You can see the details of this commit at:
http://git.savannah.gnu.org/cgit/libcdio.git/commit/?id=81d5c7bc6c162c17362d954e86de8971605d52ef
Regards,
/Pete
- [Libcdio-devel] [Pushed PATCH] Fix MinGW inlining failed warnings,
Pete Batard <=