From 1623d576465a3ed85f1567184f49390f3e530b1b Mon Sep 17 00:00:00 2001 From: Chris Forbes Date: Fri, 21 Apr 2017 13:27:08 -0700 Subject: layers: Simplify ValidateCmd Nested !=, == was needlessly weird. --- layers/core_validation.cpp | 23 +++++++++++------------ 1 file changed, 11 insertions(+), 12 deletions(-) (limited to 'layers/core_validation.cpp') diff --git a/layers/core_validation.cpp b/layers/core_validation.cpp index de5949e6..70888257 100644 --- a/layers/core_validation.cpp +++ b/layers/core_validation.cpp @@ -3510,19 +3510,18 @@ static bool ReportInvalidCommandBuffer(layer_data *dev_data, GLOBAL_CB_NODE *cb_ // Validate the given command being added to the specified cmd buffer, flagging errors if CB is not in the recording state or if // there's an issue with the Cmd ordering bool ValidateCmd(layer_data *dev_data, GLOBAL_CB_NODE *cb_state, const CMD_TYPE cmd, const char *caller_name) { - bool skip = false; - if (cb_state->state != CB_RECORDING) { - if (cb_state->state == CB_INVALID) { - skip |= ReportInvalidCommandBuffer(dev_data, cb_state, caller_name); - } else { - skip |= log_msg(dev_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_COMMAND_BUFFER_EXT, - reinterpret_cast(cb_state->commandBuffer), __LINE__, DRAWSTATE_NO_BEGIN_COMMAND_BUFFER, - "DS", "You must call vkBeginCommandBuffer() before this call to %s", caller_name); - } - } else { - skip |= ValidateCmdSubpassState(dev_data, cb_state, cmd); + switch (cb_state->state) { + case CB_RECORDING: + return ValidateCmdSubpassState(dev_data, cb_state, cmd); + + case CB_INVALID: + return ReportInvalidCommandBuffer(dev_data, cb_state, caller_name); + + default: + return log_msg(dev_data->report_data, VK_DEBUG_REPORT_ERROR_BIT_EXT, VK_DEBUG_REPORT_OBJECT_TYPE_COMMAND_BUFFER_EXT, + reinterpret_cast(cb_state->commandBuffer), __LINE__, DRAWSTATE_NO_BEGIN_COMMAND_BUFFER, "DS", + "You must call vkBeginCommandBuffer() before this call to %s", caller_name); } - return skip; } void UpdateCmdBufferLastCmd(GLOBAL_CB_NODE *cb_state, const CMD_TYPE cmd) { -- cgit v1.2.3