andy-thomason / Vookoo

A set of utilities for taking the pain out of Vulkan in header only modern C++
MIT License
522 stars 52 forks source link

Remove semantically useless const qualifiers on returns and fix examples with the cleanup. #36

Closed FunMiles closed 4 years ago

FunMiles commented 4 years ago

This should be an easy pull request. It's a simple series of clang-tidy flagged cleanups. They do create a strange issue with the examples, but it is because the examples had a confusing auto & where it should have been either const auto & or auto. All such instances have been addressed.

lhog commented 4 years ago

Looks like your version indeed produces less asm instructions: https://godbolt.org/z/6WGW7o