Hi,

This is in relation to the RT 35517 ticket at http://rt.cpan.org/Ticket/Display.html?id=35517 which seems like it should be a simple enough fix.

The suggestion there is to remove the following piece of code from Inline::install() :

   croak M64_install_not_c($o->{API}{language_id})
     unless uc($o->{API}{language_id}) =~ /^(C|CPP)$/ ;

However, on the assumption that a useful check is being carried out by that code, maybe we really ought to be changing that code to something like:

   croak M64_install_not_c($o->{API}{language_id})
unless uc($o->{API}{language_id}) =~ /^(C|CPP|Java|Python|Ruby|Lisp|Pdlpp)$/ ;

Are there any other languages (I've missed) that ought to be added to that regex ?
Are there any languages in that regex that ought to be removed ?

Or do we simply follow the advice given in the bug report ?

Cheers,
Rob

Reply via email to