epam / epam/Indigo

review all such cases where safe type conversion requires `dynamic_cast` and get rid of downcasting as far as possible.

Open
#449 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
406
Forks
134
Avg merge
2d 11h
Merged PRs (30d)
24

Description

We agreed to avoid using `dynamic_cast` because of possible performance overhead. I revoke all the affected comments.
Other hand, "need of dynamci_cast is a signal that something is wrong in the code and design.
We need to review all such cases and get rid of type downcasting as far as possible.

_Originally posted by @SunFellow in https://github.com/epam/Indigo/issues/423#issuecomment-879998306_

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.