tinygltf icon indicating copy to clipboard operation
tinygltf copied to clipboard

Published 20 hours ago • verified publisher icon syoyo

Code refactor

Open GermanAizek opened this issue 3 years ago • 6 comments

@syoyo Strings replaced by characters where possible, initializing object variables in the constructor, default constructors and destructors, using std::move, and deleting unused code.

GermanAizek avatar Oct 22 '21 15:10 GermanAizek

You should describe details of this PR

syoyo avatar Oct 23 '21 09:10 syoyo

@syoyo updated details of this PR

GermanAizek avatar Oct 23 '21 13:10 GermanAizek

Dear @syoyo, check it again.

GermanAizek avatar Oct 27 '21 07:10 GermanAizek

@GermanAizek Changes seem marginal and unimportant, so may not merge this PR

syoyo avatar Oct 27 '21 11:10 syoyo

@syoyo, review new commit again.

GermanAizek avatar Oct 27 '21 12:10 GermanAizek

@GermanAizek Please create new PR for Fixed possible overflow CWE-190

syoyo avatar Oct 27 '21 12:10 syoyo