Merge pull request #1461 from jfaust/master

Fix possible infinite loop when exporting to gltf2

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

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

Fix mesh primitive’s attributes’ names

Per spec

TEXCOORD -> TEXCOORD_0
COLOR -> COLOR_0
JOINTS -> JOINTS_0
WEIGHTS -> WEIGHTS_0

Remove JOINTMATRIX since it’s not supported (and doesn’t seem to be output, anyway)

TANGENT should be added at a later date

Remove KHR_binary_glTF code

Binary glTF is now part of the glTF2 spec. However, it’s implemented incorrectly, so will be temporarily removed

Remove compresssed file format flag

Add gltf2 basic unit test

more specific token search for Collada Loader

collada can appear in many files, such as glTFs via the “generator” field (in the form of collada2gltf)

Load gltf .bin files from correct directory

Remove premultipliedAlpha from gltf2

Fix parsing of glTF version
Handle version as int in gltf
Fix format specifiers in glTF version parser

Add support for importing both glTF and glTF2 files

Restrict search for OFF header to first 3 bytes

Remove commented out code

Set alphaMode, baseColorFactor opacity when model’s opacity isn’t 1

Set the metallicFactor to 0 if source file doesn’t have metallicFactor

Otherwise, the models have full metallic materials and appear very dim

Fix METALLIC_FACTOR typo

Define gltf material property names as constants

Fix indentation

Implement pbrSpecularGlossiness property as Nullable

Define default material values as static constants

for reuse

Remove need for Has by returning an empty Ref in Get

Use different form of index accessor

Export material names properly

Sampler improvements; Add new LazyDict method

Samplers are now imported into assimp internal format from textures;
Samplers have defaults as per spec;
Sampler enums are strongly typed with UNSET values;
Samplers are exported properly;
Sampler filters are exported as well;
Samplers are re-used across textures on export
Default sampler values are not written

Set default values on Sampler

Fix unused CopyValue

Export extensions

Formatting