On Thu, 8 Jul 2010, Jan Dubois wrote: > On Thu, 08 Jul 2010, Andy Dougherty wrote: > > On Thu, 8 Jul 2010, Marvin Humphrey wrote: > > > > > I'm very happy to see this patch, and I hope the experiment works out. > > > > > > It looks like only the variant "inline" is probed for. I have found it useful > > > to probe for "__inline" and "__inline__" first and prefer those. GCC disables > > > the "inline" keyword under "-ansi": > > > > > > http://gcc.gnu.org/onlinedocs/gcc/Alternate-Keywords.html > > > > > > Also, for MSVC, IIRC only "__inline" works. > > > > Thanks for the feedback (and the later verification that MSVC only > > documents __inline as supported). > > Note though that on older compiler versions it has only a single leading > underscore: '_inline', so we need to be able to #define it to different > values. > > But given that Perl doesn't use Configure on Windows I thought it wouldn't > matter either way. Thanks. Knowing that it will need to be redefined is useful -- I'm restructuring the patch so that such redefinition is easier to do. (This will also make it easier to catch the corner case of gcc -ansi.) -- Andy Dougherty doughera@lafayette.eduThread Previous