Re: [SEMANAGE] Swig update: typemaps

[Joshua Brindle <jbrindle@tresys.com>]

Joshua Brindle wrote:
> Ivan Gyurdiev wrote:
> 
<snip>
>> @@ -67,6 +73,14 @@
>>  %apply int *OUTPUT { int * };
>>  %apply int *OUTPUT { size_t * };
>>  
>> +%typemap(in, numinputs=0) char **(char *temp) {
>> +    $1 = &temp;
>> +}
>> +
>> +%typemap(argout) char** {
>> +    $result = t_output_helper($result, SWIG_FromCharPtr(*$1));
>> +}
>> +
> 
> 
> Have to be careful with this one, you don't want it messing with other 
> char**. Did you run all the tests in pywrap-test after rebuilding this 
> to make sure it didn't interrupt anything else?
> 
> The rest look fine.
> 

Speaking of which. Anytime you add anything to the wrapper please add 
the corresponding tests to pywrap-test. I'd like it if there was always 
a test for anything in mainline (I think Steve would too)



--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.