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

Fix warnings #1741

Merged
merged 2 commits into from
Sep 16, 2024
Merged

Fix warnings #1741

merged 2 commits into from
Sep 16, 2024

Conversation

dfriederich
Copy link
Contributor

Minor changes to reduce warnings seen when using gfxreconstruct, no (intentional) functional changes.

Using struct as in the definition for forward declaration of the ReplayDeviceInfo tag (clang -Wmismatched-tags)

Explicitly define assignment operators for clang
-Wdeprecated-copy-with-user-provided-copy warning.
clang output:

definition of implicit copy assignment operator for 'ShaderDescriptorInfo'
is deprecated because it has a user-provided copy constructor

Minor changes to reduce warnings seen when using gfxreconstruct,
no (intentional) functional changes.

Using struct as in the definition for forward declaration of the
ReplayDeviceInfo tag (clang -Wmismatched-tags)

Explicitely define assignment operators for clang
 -Wdeprecated-copy-with-user-provided-copy warning.
  clang output:
> definition of implicit copy assignment operator for 'ShaderDescriptorInfo'
> is deprecated because it has a user-provided copy constructor
@ci-tester-lunarg
Copy link

Author dfriederich not on autobuild list. Waiting for curator authorization before starting CI build.

@dfriederich
Copy link
Contributor Author

minor change, mostly to avoid getting warnings when using gfxreconstruct in a project as dependency with enabled warnings.

@ci-tester-lunarg
Copy link

Author dfriederich not on autobuild list. Waiting for curator authorization before starting CI build.

@dfriederich
Copy link
Contributor Author

@bradgrantham-lunarg thanks for the format fix :-)

@ci-tester-lunarg
Copy link

CI gfxreconstruct build queued with queue ID 255991.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4827 running.

@ci-tester-lunarg
Copy link

CI gfxreconstruct build # 4827 passed.

@bradgrantham-lunarg bradgrantham-lunarg merged commit 247ff67 into LunarG:dev Sep 16, 2024
8 checks passed
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.

3 participants