SWIG

On 27/08/12 09:01, Johan Hake wrote:
> Hello!
>
> svn revision 13637 made my present compilation setup break. I use -Wall
> -Werror for my g++ (version 4.6.3).
>
> It is changes introduced in the SWIG_InitilizeModule function, defined
> in swiginit.swg, which g++ chokes on. More specific:
>
>    SWIG_InitializeModule(void *clientdata) {
>    size_t i;
>    swig_module_info *module_head, *iter;
>    int found, init;
>    (void *)clientdata;<---
>
> With the following g++ message:
>
>    error: statement has no effect [-Werror=unused-value]
>
> when removing -Werror I only get a warning and removing -Wall it is
> again silent.
>
> The error is somewhat strange as the commit message for this revision
> (13637) indicates that the revision should fix problems with unused-values.

There have been a few attempts to squash warnings due to 'clientdata' 
sometimes not being used. The difficulty is it is used by some target 
languages but not others. Anyway, I think I have committed a good 
solution to this now, but it meant changing quite a few macros. Try it out.

William

Works like a charm!

Thanks!

Johan

On 09/07/2012 08:32 AM, William S Fulton wrote:
> On 27/08/12 09:01, Johan Hake wrote:
>> Hello!
>>
>> svn revision 13637 made my present compilation setup break. I use -Wall
>> -Werror for my g++ (version 4.6.3).
>>
>> It is changes introduced in the SWIG_InitilizeModule function, defined
>> in swiginit.swg, which g++ chokes on. More specific:
>>
>>    SWIG_InitializeModule(void *clientdata) {
>>    size_t i;
>>    swig_module_info *module_head, *iter;
>>    int found, init;
>>    (void *)clientdata;<---
>>
>> With the following g++ message:
>>
>>    error: statement has no effect [-Werror=unused-value]
>>
>> when removing -Werror I only get a warning and removing -Wall it is
>> again silent.
>>
>> The error is somewhat strange as the commit message for this revision
>> (13637) indicates that the revision should fix problems with
>> unused-values.
> 
> There have been a few attempts to squash warnings due to 'clientdata'
> sometimes not being used. The difficulty is it is used by some target
> languages but not others. Anyway, I think I have committed a good
> solution to this now, but it meant changing quite a few macros. Try it out.
> 
> William