Some of our static analysis tools were complaining about unreachable code due to "#if ... return false; #endif code " blocks. This was changed to "#if ... return false #else ... #endif" which doesn't change the functionality but makes the static analyzers happy.
Also removed BASISU_NOTE_UNUSED on two variables that are no longer declared (resulting in a build error when BASISD_SUPPORT_UASTC was not defined)
Encoder is now a library in the "encoder" directory
JavaScript wrappers now expose the entire codec: encoder, transcoder, container independent transcoding, and .basis file information
This PR simply wildcards files in the repository in the metainformation
.reuse/dep5 to express copyright and licensing information for files
that can't easily be parsed by REUSE. If any of the assertions made in
.reuse/dep5 are incorrect, we should fix that before this PR
is accepted.
This approach works OK, though a more intrusive, but superior version of
this PR would simplify copyright / license statements as follows, and
make sure they're present in every file:
// Copyright 2019-2020 Binomial LLC
// SPDX-License-Identifier: Apache-2.0
(or whatever the appropriate copyright owner and license are for each
file). If that were done, the wildcarding in .reuse/dep5 would not be
required to pass REUSE.
(1 << num_bits) was converted to a signed integer that caused an overflow (as num_bits was usually set to 31). It worked fine since the value was later converted back to uint32_t but signed integer overflow is technically an undefined behavior and it was triggering errors in our automated tests.