Fix delete / delete[] mismatches in glTF2 importer

Fix delete / delete[] mismatches in MakeVerboseFormat

Fix out-of-bounds read in MaterialSystem unit test

Fix delete / delete[] mismatch in glTFAsset

Enable AddressSanitizer for Linux clang build

Add AddressSanitizer option to CMake

Merge pull request #1466 from jaredmulconry/issue_1330

Eliminated warnings when building the project with clang

Merge pull request #1465 from turol/warnings

Make Travis treat warnings as errors

travis: Treat warnings as errors

Fix CMAKE option name

Open3DGC: Fix some uninitialized variable warnings

Eliminated all warnings under clang with default settings. One remains in the included zlib contrib project.

Added -fPIC flag to C compilers for GCC and clang. Removed -pedantic flag from some compilers.

add some asserts.

closes https://github.com/assimp/assimp/issues/1450: use correct name of exporter to gltf2

Obj: rename attribute from exporter.

Merge branch 'master' of https://github.com/assimp/assimp

closes https://github.com/assimp/assimp/issues/1459: fix out-of-boundary access error

Merge pull request #1462 from jaredmulconry/issue_1330

Fully addressed identifiable build warnings on gcc

Merge branch 'master' of github.com:assimp/assimp into issue_1330

Merge pull request #1461 from jfaust/master

Fix possible infinite loop when exporting to gltf2

Changed a couple more function interfaces to correct the input type.

Replaced unsigned long for the crc table to z_crc_t, to match what is returned by get-crc_table

Fix glTF2::Asset::FindUniqueID() when the input string is >= 256 chars

Improved the naming of temporary file generator function. Replaced use of tmpnam in utIOStreamBuffer with this facility to addresssafety warning.

Added a header to hold the file generation code for unit testing purposes.

Merge branch 'master' of github.com:assimp/assimp into issue_1330

Changed the method by which temporary files are created for unit the FileSizeTest. Will apply to other tests next.

Merge pull request #1457 from jaredmulconry/issue_1330

Partially address issue #1330

Replaced flakey macros with specific functions to serve the purpose

Changed the FileSizeTest to not rely on tmpnam to eliminate warning on gcc.

Suppressed warning on gcc caused by the 'visibility' attribute being ignored on types.

Fixed unused variable warning by replacing them with descriptive comments

Fixed a warning caused by aiVector3D appearing in a packed struct, causing it to fail to pack as requested.

Fixed an error when compiling samples under MSVC that was caused by assuming including windows.h would pull in shellapi.h

Addressed warnings generated on MSVC across x86 and x64.

Merge pull request #1444 from turol/warnings

Clean up some more GCC warnings

Merge pull request #1445 from Matter-and-Form/gltf2-alphaMode-fix

Fix glTF2 alphaMode storage and reading

Merge pull request #1447 from Matter-and-Form/gltf1-color-import-fix

[gltf1] Don’t ignore rgba(1,1,1,1) color properties

Merge pull request #1446 from Matter-and-Form/feature/gltf2-primitives

glTF2 primitives fixes

Merge branch 'master' of https://github.com/assimp/assimp

FBX: add missing inversion of postrotation matrix for fbx.

Don’t ignore rgba(1,1,1,1) color properties

Do not ignore rgba(1,1,1,1) material properties when importing glTF1. While a white diffuse color may be the default value for a default assimp material, `1,1,1,1` is a very explicit color value for ambient/specular/emissive color properties.

Closes #1434

Formatting

Fix alphaMode storage and reading

alphaMode is now converted from a std::string to an aiString and back to std::string, since aiString is easier to store and retrieve from aiMaterial properties than std::string

Fixes issues of alphaMode being written out as `\fOPAQUE\0\0\0\0\0\0…`

SImplify mesh merging code

My assumption that primitives of different types (modes) can’t be in the same mesh was incorrect.

tweaks to primitive merging logic; comments + formatting

Formatting

Merge multiple meshes in a node into one mesh with many primtives; write out only one mesh per node

To do:
- clean up MergeMeshes
- see if there’s a way to do this earlier in the flow

Revert "store node mesh vs. meshes"

This reverts commit a0d97505e5a53a9aa3a734d6bd3ad790234e6e09.

Merge pull request #1442 from jcowles/master

Fix glTF 2.0 multi-primitive support

BlenderDNA: Silence warning about inline function which is declared but not defined

It's a templated method which is meant to be specialized. The base version
is never called. Just remove 'inline' to make GCC shut up.

glTF: Silence uninitialized variable warning

This is a false positive. 'jointNamesIndex' is either set by the loop
or the following conditional is false which also sets it. The undefined value
is never seen by the following code.

PlyLoader: Fix operator precedence issue in header check

The previous version might read past end of buffer

FIReader: Silence uninitialized variable warning

This is a false positive. First time through the loop 'imod3' is always 0
so c1 is not used. It's also set so further iterations have a valid 'c1'.
If 'value' is empty the switch doesn't look at 'c1' either since 'imod3'
is still 0.

Collada: Silence uninitialized variable warning

This is a false positive. Value of 'method' is only used if 'targetMeshes'
contains something and all paths through the first loop which add stuff
to 'targetMeshes' also set 'method'.

Fix glTF 2.0 multi-primitive support

Previously, only one primitive was supported, in fact memory was corrupted
when more than one primitive was found per glTF mesh.

In this change, each primitive is unrolled as a new Assimp Mesh, resulting
in multiple Assimp meshes per node when multiple primitives exist per
glTF mesh. This is required in the general case, since glTF primitives can
have different material bindings and primitive modes.

Merge pull request #1441 from turol/travis

Disable Travis OS X build since it doesn't do anything currently

Travis: Disable OS X build since it doesn't do anything currently

Merge pull request #1440 from turol/travis

Travis cleanup

Merge pull request #1435 from jaredmulconry/issue_1065

Last remaining build warning emitted from MSVC for x64 builds

Merge pull request #1437 from rmitton/sib-version

Added support for SIB models from Silo 2.5

travis: Enable ccache

travis: Only build with xcode 8.3

travis: Correctly minimize build matrix

travis: Move os declarations earlier

Merge pull request #1436 from turol/warnings

Clean up some GCC warnings

Added Silo 2.5 support

Silo 2.5 bumps the version number of SIB files for no apparent reason. Doesn't appear to be any other changes to the file format.

Fixed truncated material names

The previous Unicode library change accidentally trimmed off the last character of SIB material names.

Merge pull request #1 from assimp/master

Merged from master

FBXParser: Silence uninitialized variable warnings

Importer: Whitespace cleanup to fix GCC misleading indentation warning

AssbinExporter: Fix strict aliasing violation

AssbinExporter: Add Write specialization for aiColor3D

DefaultLogger: Whitespace cleanup to fix GCC misleading indentation warning

glTFExporter: Silence uninitialized variable warning

This is a false positive, idx_srcdata_begin is only used if comp_allow
is true and in that case it's also initialized.

Add CMake flag to treat warnings as errors

Addressed last remaining warning under MSVC caused by use of 'deprecated' fopen.

Merge pull request #1433 from vkovalev123/patch-1

Update 3DSLoader.cpp

Update 3DSLoader.cpp

Fixed reading of CHINK_RGBF. If reading performs on x32 platform then reading will execute right, but on x64 it`s wrong because it will read 8 bytes instead 4.

Merge pull request #1431 from assimp/revert-1427-asan

Revert "Asan"

Revert "Asan"

Merge pull request #1423 from Matter-and-Form/feature/gltf2

glTF 2 Importer + glTF 2 Spec Conformity

Merge pull request #1427 from turol/asan

Asan

Update glTF in list of importers and exporters

Fix delete / delete[] mismatch in glTFAsset

Enable AddressSanitizer for Linux clang build

MDP: fix encoding issues.

Merge branch 'master' of https://github.com/assimp/assimp

FBX: fix some minor findings.

closes https://github.com/assimp/assimp/issues/1426: add Defines.h to include folder for install.

Add AddressSanitizer option to CMake

Manually read alphaMode material property

Properly move string passed to JSON writer

Fix Segfault caused by losing pointer to std::string

Keep std::string alive

Revert "Remove simple gltf2 export unit test"

This reverts commit 4b01ecaf109ae1bbd3ae918e30abdc67d7448a5a.

Remove simple gltf2 export unit test

Segfaults on Linux for some reason. No other tests test exporting, so it’s fine

Exclude glTF2 Exporter test when ASSIMP_BUILD_NO_EXPORT

Formatting

Use `forceNumber` argument of `WriteAttrs` to write correct attribute names, instead