Archived issue #0032906

Coding Rules - get rid of std::iterator inheritance (deprecated since C++17)

Open CASCADEOCCT:Codingverified11 public notes

Search issues

Description

Get rid of std::iterator inheritance (deprecated since C++17).

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Dmitry DZAMA
Date: Wed Mar 30 09:24:49 2022 +0300

    0032906: Get rid of std::iterator inheritance (deprecated since C++17).
02Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
03Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
07Commenter 2
Commenter 3, could you please take a look into this patch>?
08Commenter 8
I have no remarks, provided that this change does not break compatibility with older compilers that we are still supporting
09Commenter 9
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 18635.800000000607 / 18527.100000000686 [+0.59%]
Products
Total CPU difference: 11716.970000000136 / 11769.420000000111 [-0.45%]
Windows-64-VC14:
OCCT
Total CPU difference: 20692.578125 / 20548.828125 [+0.70%]
Products
Total CPU difference: 13247.5625 / 13272.65625 [-0.19%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
10Commenter 1
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
11Commenter 1
Branch [archived branch] has been deleted by Participant.

[revision removed]

Related records