[vtk-developers] VTK Coding Standards, C++11 override and =delete

[Sean McBride]

On Fri, 6 May 2016 11:42:29 -0400, Marcus D. Hanwell said:

>> The proposal is:
>>
>> * Use "virtual" on the original base class declaration of the method.
>>
>> * Do not use "virtual" in any derived class declaration of the method.
>>
>> * Use "override" on all derived classes.  Leave this off in C++03
>>   by doing it through a macro.
>>
>> This will work in C++03, C++11, etc.
>>
>That is exactly what I was proposing - thanks Brad.

Well, after a week I haven't heard any disagreements, so I updated my patch to remove the virtual keyword when adding override:

<https://gitlab.kitware.com/vtk/vtk/merge_requests/1458>

If there is consensus, I'd appreciate a few +2's, then I'll merge and watch the dashboards.

Cheers,

-- 
____________________________________________________________
Sean McBride, B. Eng                 sean at rogue-research.com
Rogue Research                        www.rogue-research.com 
Mac Software Developer              Montréal, Québec, Canada