Skip to content

Remove using namespace ov::op and ov::pass::pattern directives to fix UNITY build conflicts#33134

Merged
mryzhov merged 15 commits intoopenvinotoolkit:masterfrom
evkotov:CVS-177276
Jan 15, 2026
Merged

Remove using namespace ov::op and ov::pass::pattern directives to fix UNITY build conflicts#33134
mryzhov merged 15 commits intoopenvinotoolkit:masterfrom
evkotov:CVS-177276

Conversation

@evkotov
Copy link
Copy Markdown
Contributor

@evkotov evkotov commented Dec 5, 2025

Add using declarations and namespace aliases to improve code readability after removing using namespace directives.

Details:

Remove using namespace directives that cause name conflicts in UNITY builds and replace them with explicit using declarations and namespace aliases for improved readability.

  • Remove using namespace ov::op and using namespace ov::pass::pattern directives
  • Add specific using declarations (e.g., using ov::pass::pattern::wrap_type;) only when a construct is used 2+ times in a file
  • Add namespace aliases (e.g., namespace v0 = ov::op::v0;) only when a version namespace is used 2+ times in a file
  • Replace fully qualified names with short forms where declarations are added

Tickets:

  • 177276

@evkotov evkotov self-assigned this Dec 5, 2025
@evkotov evkotov requested a review from a team as a code owner December 5, 2025 11:22
@evkotov evkotov added the category: transformations OpenVINO Runtime library - Transformations label Dec 5, 2025
@evkotov evkotov force-pushed the CVS-177276 branch 2 times, most recently from 41d889a to 42513d4 Compare December 17, 2025 12:37
@praasz praasz self-assigned this Dec 17, 2025
Copy link
Copy Markdown
Contributor

@praasz praasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added comments to consider

@praasz praasz added this to the 2026.0 milestone Jan 13, 2026
@evkotov evkotov requested a review from a team as a code owner January 14, 2026 09:44
@github-actions github-actions bot added the category: JS API OpenVino JS API Bindings label Jan 14, 2026
@github-actions github-actions bot removed the category: JS API OpenVino JS API Bindings label Jan 14, 2026
@evkotov evkotov requested a review from praasz January 14, 2026 17:03
@mryzhov mryzhov added this pull request to the merge queue Jan 15, 2026
github-merge-queue bot pushed a commit that referenced this pull request Jan 15, 2026
…x UNITY build conflicts (#33134)

Add using declarations and namespace aliases to improve code readability
after removing using namespace directives.

### Details:
Remove using namespace directives that cause name conflicts in UNITY
builds and replace them with explicit using declarations and namespace
aliases for improved readability.
- Remove using namespace ov::op and using namespace ov::pass::pattern
directives
- Add specific using declarations (e.g., using
ov::pass::pattern::wrap_type;) only when a construct is used 2+ times in
a file
- Add namespace aliases (e.g., namespace v0 = ov::op::v0;) only when a
version namespace is used 2+ times in a file
- Replace fully qualified names with short forms where declarations are
added

### Tickets:
 - 177276
Merged via the queue into openvinotoolkit:master with commit 7859e26 Jan 15, 2026
218 of 222 checks passed
@evkotov evkotov deleted the CVS-177276 branch January 15, 2026 17:45
Naseer-010 pushed a commit to Naseer-010/openvino that referenced this pull request Feb 18, 2026
…x UNITY build conflicts (openvinotoolkit#33134)

Add using declarations and namespace aliases to improve code readability
after removing using namespace directives.

### Details:
Remove using namespace directives that cause name conflicts in UNITY
builds and replace them with explicit using declarations and namespace
aliases for improved readability.
- Remove using namespace ov::op and using namespace ov::pass::pattern
directives
- Add specific using declarations (e.g., using
ov::pass::pattern::wrap_type;) only when a construct is used 2+ times in
a file
- Add namespace aliases (e.g., namespace v0 = ov::op::v0;) only when a
version namespace is used 2+ times in a file
- Replace fully qualified names with short forms where declarations are
added

### Tickets:
 - 177276
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category: transformations OpenVINO Runtime library - Transformations Code Freeze

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants