Added copy/move constructors of complex RenderPass types.

gpu/vulkan/vulkan_render_pass.cc

Index: gpu/vulkan/vulkan_render_pass.cc
diff --git a/gpu/vulkan/vulkan_render_pass.cc b/gpu/vulkan/vulkan_render_pass.cc
index a142fc1b3ee6d38d03ad805f76c1aa915416c60a..da469d0ad96479a003fda86dce6cd7ec784ba370 100644
--- a/gpu/vulkan/vulkan_render_pass.cc
+++ b/gpu/vulkan/vulkan_render_pass.cc
@@ -99,6 +99,12 @@ bool VulkanRenderPass::AttachmentData::ValidateData(
 
 VulkanRenderPass::SubpassData::SubpassData() {}
 
+VulkanRenderPass::SubpassData::SubpassData(const SubpassData& data)

[piman, 2016/04/22 18:23:03]

nit: you can use '= default' for maintenance (if we add fields etc.).

[David Yen, 2016/04/22 18:31:18]

Done.

+    : subpass_attachments(data.subpass_attachments) {}
+
+VulkanRenderPass::SubpassData::SubpassData(SubpassData&& data)
+    : subpass_attachments(std::move(data.subpass_attachments)) {}

[piman, 2016/04/22 18:23:03]

ditto

[David Yen, 2016/04/22 18:31:18]

Done.

+
 VulkanRenderPass::SubpassData::~SubpassData() {}
 
 bool VulkanRenderPass::SubpassData::ValidateData(
@@ -126,6 +132,14 @@ bool VulkanRenderPass::SubpassData::ValidateData(
 
 VulkanRenderPass::RenderPassData::RenderPassData() {}
 
+VulkanRenderPass::RenderPassData::RenderPassData(const RenderPassData& data)
+    : attachments(data.attachments),
+      subpass_datas(data.subpass_datas) {}

[piman, 2016/04/22 18:23:59]

Both of these as well.

[David Yen, 2016/04/22 18:31:19]

Done.

+
+VulkanRenderPass::RenderPassData::RenderPassData(RenderPassData&& data)
+    : attachments(std::move(data.attachments)),
+      subpass_datas(std::move(data.subpass_datas)) {}
+
 VulkanRenderPass::RenderPassData::~RenderPassData() {}
 
 VulkanRenderPass::VulkanRenderPass(VulkanDeviceQueue* device_queue)