syoyo / tinygltf

Header only C++11 tiny glTF 2.0 library
MIT License
2.04k stars 413 forks source link

Modernize #433

Closed dyollb closed 1 year ago

dyollb commented 1 year ago

This pull request modernizes the code a bit, replacing member initialization lists by default member initialization. This is related to https://github.com/syoyo/tinygltf/commit/7a570c88d996918489bbbd3af7ebbda1f3f74a0a#r119070408

I used clang-tidy to refactor the code. The following checks were used:

syoyo commented 1 year ago

Nice! Each changes seems trivial, but lots of changes... Need some time to review it and ensure no side-effect exists.

syoyo commented 1 year ago

Run some tests and seems no side-effects observed, so merged!