Vulkan-Cookbook icon indicating copy to clipboard operation
Vulkan-Cookbook copied to clipboard

Update 19 Increasing the performance through increasing the number of…

Open richardebel opened this issue 2 years ago • 1 comments

… separately rendered frames.h

Changed 5 references to r-value references to fix build errors for mingw64-gcc and VS std c++20

richardebel avatar Aug 20 '22 17:08 richardebel

A few of questions:

  1. What errors were you getting?
  2. l-value references and r-value references are two very different things. I can't see how changing the function signature is a fix. How do you know there is not a better fix?
  3. If you were getting build errors, how did you verify faster performance from just using l-value references if the code refused to build? This pull request looks like it is for two different reasons that don't quite make sense.
  4. This is not c++20 code. Why are you building with the c++20 standard? The existing standard in CMakeTests.txt clearly states c++11.

ButchDean avatar Jan 19 '23 03:01 ButchDean