[Insight-developers] Re: Controlling number of lines with CabeSwig (was Re: gcc-3.3 : final_scan_insn, at final.c:2117)

[Luis Ibanez]

Gaetan,


Thanks for this new patch.
The patch has been committed.


However, I'm afraid I missed the discussion that led to this patch.

Just for the record, could you please give us some background ?


   Thanks



       Luis



-----------------------
Gaëtan Lehmann wrote:
> 
> Le 12 sept. 07 à 11:58, Mathieu Malaterre a écrit :
> 
>> Hi Luis,
>>
>>   I believe the following patch is completely backward compatible and
>> should be completely invisible to most user (you even save some disk
>> space).
> 
> 
> I do agree. This flag has been tried before, and disabled because  
> reading the produced code with it is nearly impossible - not really  
> handy while debugging typemaps.
> I believe it can be used now that the typemaps have proved to be  usable 
> for a long enough time.
> 
> Gaëtan
> 
>>
>>   Could you please apply that before the next ITK release.
>>
>> Thanks to gaetan for the patch,
>> -Mathieu
>>
>> On 9/12/07, Gaëtan Lehmann <gaetan.lehmann at jouy.inra.fr> wrote:
>>
>>>
>>> Hi Mathieu,
>>>
>>> Le 12 sept. 07 à 11:47, Mathieu Malaterre a écrit :
>>>
>>>> The trick is to simply add -fcompact. The time to compile the  file is
>>>> exactly the same but at least I am now pleasing gcc 3.3, and  manage to
>>>> compile the object file.
>>>>
>>>
>>> That's a good news :-)
>>>
>>>> Now I need to figure out how to add this -fcompact to all target,
>>>> geatan any suggestion ?
>>>
>>>
>>> yes, you can add it in CreateWrapperLibrary.cmake, line 416
>>>
>>>    ADD_CUSTOM_COMMAND(
>>>     SOURCE ${input_idx}
>>>     COMMAND ${CSWIG}
>>>     ARGS ${swig_libs}
>>>          -I${CSWIG_DEFAULT_LIB}
>>>          ${CSWIG_IGNORE_WARNINGS}
>>>          -noruntime
>>>         -fcompact <------------ HERE !
>>>          ${cindex}
>>>          -depend ${input_xml}.depend
>>>          -outdir ${outdir}
>>>          -o ${output_cxx}
>>>          -c++
>>>          ${CSWIG_ARGS_${language}}
>>>          ${extra_args}
>>>          ${input_xml}
>>>     TARGET ${library_name}
>>>
>>> Gaëtan
>>>
>>>
>>>
>>> -- 
>>> Gaëtan Lehmann
>>> Biologie du Développement et de la Reproduction
>>> INRA de Jouy-en-Josas (France)
>>> tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
>>> http://voxel.jouy.inra.fr
>>>
>>>
>>>
>>>
>>>
>>
>>
>> -- 
>> Mathieu
> 
> 
> -- 
> Gaëtan Lehmann
> Biologie du Développement et de la Reproduction
> INRA de Jouy-en-Josas (France)
> tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
> http://voxel.jouy.inra.fr
> 
> 
>