libcdio-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]