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

Change Some Warning/Error Messages to Debug Messages #162

Merged
merged 2 commits into from
Feb 21, 2019

Conversation

MarkY-LunarG
Copy link
Contributor

@MarkY-LunarG MarkY-LunarG commented Jan 17, 2019

  • Changed VK_INCOMPLETE error messages to debug messages.
  • Changed handle mapping failure messages from warning to debug messages.

Replay can produce VK_INCOMPLETE errors and handle mapping errors due to the issues described in #189 and #187. These errors do not impact replay and should be safe to ignore. Changing the type of log message used to report them to debug to reduce log noise, until the associated issues are fixed.

@MarkY-LunarG MarkY-LunarG force-pushed the marky_master_fix_vulkaninfo_error branch 2 times, most recently from 5e77251 to 9207cb8 Compare January 17, 2019 22:32
@MarkY-LunarG MarkY-LunarG force-pushed the marky_master_fix_vulkaninfo_error branch from 9207cb8 to 7401c24 Compare January 18, 2019 20:14
MarkY-LunarG and others added 2 commits February 20, 2019 19:29
If both XCB and Wayland are present, the replay tool only uses
one.  This caused one of the Enum type queries to get a different
count and return VK_INCOMPLETE.  Now printing a debug message
instead of a warning message for VK_INCOMPLETE, as the replay tool
does not use the values returned by the query and the error should
be safe to ignore.
Changing the log message type for a handle mapping failure
from warning to debug.
@dustin-lunarg dustin-lunarg force-pushed the marky_master_fix_vulkaninfo_error branch from 7401c24 to 5d4d21f Compare February 21, 2019 02:38
@dustin-lunarg dustin-lunarg changed the title Fix VulkanInfo error on Wayland/XCB Change Some Warning/Error Messages to Debug Messages Feb 21, 2019
@dustin-lunarg
Copy link
Contributor

I have modified this PR by altering the existing commit to change the log type from warning to debug, adding a new commit to change the handle mapping failure log type to debug, and to update the PR subject and summary to reflect the purpose for these changes.

@dustin-lunarg dustin-lunarg merged commit ab485d2 into master Feb 21, 2019
@dustin-lunarg dustin-lunarg deleted the marky_master_fix_vulkaninfo_error branch February 21, 2019 20:35
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.

None yet

3 participants