Skip to content

C4250#108

Open
dirk-zimoch wants to merge 2 commits into
epics-base:masterfrom
dirk-zimoch:C4250
Open

C4250#108
dirk-zimoch wants to merge 2 commits into
epics-base:masterfrom
dirk-zimoch:C4250

Conversation

@dirk-zimoch

Copy link
Copy Markdown
Contributor

Silence Windows warning C4250 „inheritance via dominance“.

include\pv\pvData.h(539,1): warning C4250: 'epics::pvData::PVString':
inherits 'epics::pvData::PVScalarValue<std::string>::epics::pvData::PVScalarValue<std::string>::deserialize'
via dominance
@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@AppVeyorBot

Copy link
Copy Markdown

@ralphlange ralphlange 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.

LGTM

@anjohnson

Copy link
Copy Markdown
Member

Awaiting a conclusion from the discussion on epics-base#927, since these are connected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants