Fix warning about non-constant array size

CMake: Remove OpenMP stuff, it's unused and breaks Travis clang build

Travis: Treat warnings as errors, without typos this time

Travis: Build with clang too

Merge pull request #1476 from assimp/revert-1471-clang

Revert "WIP: Enable Travis clang build"

Revert "WIP: Enable Travis clang build"

Merge pull request #1474 from jaredmulconry/issue_1470

Update zlib contribution

Merge pull request #1475 from jaredmulconry/master

Fixed warning on clang 3.9 caused by static data member in class template

Merge pull request #1471 from turol/clang

WIP: Enable Travis clang build

Fixed warnings on MSVC caused by implicit conversions from double to float.

Address warnings on Clang 3.9 caused by having a static data member in a class template.

OpenGEXImporter: Copy materials to scene

OpenGEXImporter: Store RefInfo in unique_ptr so they get automatically cleaned up

OpenGEXImporter: Fix IOStream leak

OpenGEXImporter: Store ChildInfo in unique_ptr so they get automatically cleaned up

AMFImporter: Fix memory leak

UnrealLoader: Fix IOStream leak

utRemoveVCProcess: Fix memory leak

utMetadata: Fix memory leak

Upgrade RapidJSON to get rid of a clang warning

FBXImporter: Add explicit instantiation of log_prefix so other FBX source files can see it

IFCImporter: Add explicit instantiation of log_prefix so IFCMaterial.cpp can see it

Removed unnecessary files from zlib contribution

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

Fix warning about non-constant array size

CMake: Remove OpenMP stuff, it's unused and breaks Travis clang build

Travis: Treat warnings as errors, without typos this time

Travis: Build with clang too

Merge pull request #1432 from turol/asan

Asan

Merge pull request #1469 from turol/warnings

Treat warnings as errors on Appveyor

Upgraded zlib to 1.2.11.1 from the develop branch.

Merge pull request #1468 from assimp/issue_1467

closes https://github.com/assimp/assimp/issues/1467.

Disable warning 4351 on MSVC 2013

appveyor: Treat warnings as errors

Fix warnings-as-errors flag on MSVC

Addressed some mismatched news/deletes caused by the new glTF2 sources.

Addressed a number of memory leaks identified in unit tests by asan

Addressed asan failures caused by misuse of APIs within unit tests.

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

OpenGEX: improve logging to be able to detect error-prone situations.

closes https://github.com/assimp/assimp/issues/1467.

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