Re: [eigen] std::vector specialization

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


2010/1/28 Hauke Heibel <hauke.heibel@xxxxxxxxxxxxxx>:
> In order to silence the warnings we are seeing in the dashboard
> (warning: anonymous variadic macros were introduced in C99) we can add
>
>  -Wno-variadic-macros
>
> which works for GCC 4.x and according to this thread
> http://www.gamedev.net/community/forums/topic.asp?topic_id=510702, we
> might add
>
>  #pragma GCC system_header
>
> to the StdVector header when GCC 3.x is used.
>
> Does that sound all right to everybody?

In order to do  -Wno-variadic-macros for the user, we have to do a
pragma. Then re-enable it with opposite pragma. I'm not opposed to
doing it, it could go into (Disable|Enable)MSVCWarnings.h which could
get renamed to (Disable|Enable)Warnings.h.

Benoit

>
> - Hauke
>
> On Tue, Jan 26, 2010 at 10:26 PM, Gael Guennebaud
> <gael.guennebaud@xxxxxxxxx> wrote:
>> On Tue, Jan 26, 2010 at 7:57 PM, Hauke Heibel
>> <hauke.heibel@xxxxxxxxxxxxxx> wrote:
>>> On Tue, Jan 26, 2010 at 3:16 PM, Thomas Capricelli
>>> <orzel@xxxxxxxxxxxxxxx> wrote:
>>>>
>>>> We can fix this once pushed, do not worry :)
>>>
>>> I just would like hear Gael again. There was some discussion on this
>>> topic in the past, just check here:
>>>
>>> http://listengine.tuxfamily.org/lists.tuxfamily.org/eigen/2009/04/msg00143.html
>>>
>>> It seems as if there was an issue with template arguments. It might be
>>> the case that it's working here because I am using typedefs only. I
>>> have to double check that on another day.
>>
>> the problem was that we wanted to allow partial specializations in
>> order to handle Matrix* in one shot. But here you are requiring full
>> specialization for the value type, so no such problem.
>>
>> gael
>>
>>>
>>> - Hauke
>>>
>>>
>>>
>>
>>
>>
>
>
>



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/