Add the fixes we don't have from here: https://github.com/rdb/fcollada/commit/f6186c5e9258106dbe82120a359c540876016f9e
Update the includes that didn't match the ones in the source folder.
Do we have to update our fcollada? Especially if it has useful changes.
If we have to, then my comments don't make sense. We just follow fcollada changes then.
Why uint64? The size_t isn't 64 bits always.
I'd prefer to name a variable without leading underscores. This type of naming reserved for a global scope and for all scopes with a following capital letter.
This one and the same below fixes N6 from #5288.
No need to have a space after defined.
Updating Fcollada is not that important. Nevertheless though our debug build is broken on windows because of it and the game doesn't work with Fcollada compiled with vs2015 instant default.
So I thought we might as well take all the fixes from the other forks over the internet.
Fcollada is not being maintained anymore. I believe we are the only ones maintaining it
Ok. some of my comments make sense then :)
Could you attach the errors?
By the way, the game is compiled for me without the patch with VS2013. Probably we need to test the patch for VS2013 too until we drop its support.