diff options
| author | Sybren A. Stüvel <sybren@blender.org> | 2020-08-07 18:59:12 +0300 |
|---|---|---|
| committer | Sybren A. Stüvel <sybren@blender.org> | 2020-08-07 18:59:26 +0300 |
| commit | ad6e63cb96a497721d7ad6aeb5cf119d3264036c (patch) | |
| tree | 49f7a34b908d461e5584757956cba08890963d2e /source/blender/freestyle/intern | |
| parent | dcff28e1e7a8b1be9bb6acec24af7ca6f6468e77 (diff) | |
Cleanup: enable Clang-Tidy `bugprone-parent-virtual-call` rule
I added a single `NOLINT` exception with explanation.
No functional changes.
Diffstat (limited to 'source/blender/freestyle/intern')
| -rw-r--r-- | source/blender/freestyle/intern/scene_graph/OrientedLineRep.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source/blender/freestyle/intern/scene_graph/OrientedLineRep.cpp b/source/blender/freestyle/intern/scene_graph/OrientedLineRep.cpp index 9e83527acad..e8e21d37d00 100644 --- a/source/blender/freestyle/intern/scene_graph/OrientedLineRep.cpp +++ b/source/blender/freestyle/intern/scene_graph/OrientedLineRep.cpp @@ -27,7 +27,9 @@ namespace Freestyle { void OrientedLineRep::accept(SceneVisitor &v) { - Rep::accept(v); + Rep::accept(v); // NOLINT(bugprone-parent-virtual-call), this seems to intentionally *not* call + // the parent class' accept() function, but rather the grandparent's. The + // v.visitLineRep(*this); call below is actually what the parent class would do. if (!frs_material()) { v.visitOrientedLineRep(*this); } |