Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stratimikos: "declaration shadows a typedef" warnings from Apple clang 10.0.0 #4300

Closed
CamelliaDPG opened this issue Jan 30, 2019 · 2 comments
Labels
ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates client: ATDM Any issue primarily impacting the ATDM project client: EMPIRE All issues that most directly target the ATDM EMPIRE code CLOSED_DUE_TO_INACTIVITY Issue or PR has been closed by the GitHub Actions bot due to inactivity. MARKED_FOR_CLOSURE Issue or PR is marked for auto-closure by the GitHub Actions bot. pkg: Stratimikos

Comments

@CamelliaDPG
Copy link
Contributor

@trilinos/stratimikos

Expectations

Builds with clang that use the -Wshadow flag should not emit warnings.

Current Behavior

One warning is emitted by clang when building BelosThyraAdapter.hpp.

.../include/BelosThyraAdapter.hpp:301:49: warning: declaration shadows a typedef in 'MultiVecTraits<type-parameter-0-0, MultiVectorBase<type-parameter-0-0> >' [-Wshadow]
      typedef Teuchos::ScalarTraits<ScalarType> ST;
                                                ^
.../include/BelosThyraAdapter.hpp:87:47: note: previous declaration is here
    typedef Teuchos::ScalarTraits<ScalarType> ST;

Possible Solution

Since the shadowing typedef is identical to the shadowed typedef, likely the correct solution is simply to delete the shadowing typedef (delete line 301).

Environment

I'm building on a Mac using Apple clang 10.0.0.

@CamelliaDPG CamelliaDPG added pkg: Stratimikos client: ATDM Any issue primarily impacting the ATDM project client: EMPIRE All issues that most directly target the ATDM EMPIRE code ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates labels Jan 30, 2019
@github-actions
Copy link

This issue has had no activity for 365 days and is marked for closure. It will be closed after an additional 30 days of inactivity.
If you would like to keep this issue open please add a comment and/or remove the MARKED_FOR_CLOSURE label.
If this issue should be kept open even with no activity beyond the time limits you can add the label DO_NOT_AUTOCLOSE.
If it is ok for this issue to be closed, feel free to go ahead and close it. Please do not add any comments or change any labels or otherwise touch this issue unless your intention is to reset the inactivity counter for an additional year.

@github-actions github-actions bot added the MARKED_FOR_CLOSURE Issue or PR is marked for auto-closure by the GitHub Actions bot. label Aug 18, 2021
@github-actions
Copy link

This issue was closed due to inactivity for 395 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ATDM Sev: Nonblocker Problems with Trilinos that should not block ATDM APPs from getting updates client: ATDM Any issue primarily impacting the ATDM project client: EMPIRE All issues that most directly target the ATDM EMPIRE code CLOSED_DUE_TO_INACTIVITY Issue or PR has been closed by the GitHub Actions bot due to inactivity. MARKED_FOR_CLOSURE Issue or PR is marked for auto-closure by the GitHub Actions bot. pkg: Stratimikos
Projects
None yet
Development

No branches or pull requests

1 participant