Skip to content

Add alias self-assignment CopyableAtomic test#670

Closed
AAmbuj wants to merge 1 commit into
eclipse-score:mainfrom
AAmbuj:amsh_fix_copyable_atomic_self_assign_test
Closed

Add alias self-assignment CopyableAtomic test#670
AAmbuj wants to merge 1 commit into
eclipse-score:mainfrom
AAmbuj:amsh_fix_copyable_atomic_self_assign_test

Conversation

@AAmbuj

@AAmbuj AAmbuj commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

its showing warrning as error. cause the build fail. so, i fixed it.

@castler castler left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We should rather ignore the warning Here, as the Test was s self assignment specificaly

@AAmbuj

AAmbuj commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

Hi @castler going this way is ok?

#pragma clang diagnostic push
#pragma clang diagnostic ignored "-Wself-assign-overloaded"
unit = unit;
#pragma clang diagnostic pop

or

i remove target it self
features = COMPILER_WARNING_FEATURES + ["-score_communication_treat_warnings_as_errors"],

@AAmbuj

AAmbuj commented Jul 9, 2026

Copy link
Copy Markdown
Contributor Author

I modified target.

@AAmbuj AAmbuj closed this Jul 9, 2026
@github-project-automation github-project-automation Bot moved this from Backlog to Done in COM - Communication FT Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants