Merge pull request #551 from mayeut/coc-qcc

Add COC/QCC in main header when needed

Merge pull request #690 from rouault/failed_malloc_opj_dwt_encode_procedure

[git/2.1 regression] Fix opj_write_tile() failure when numresolutions=1

Add comment explaining bj is not use when l_data_size == 0

[git/2.1 regression] Fix opj_write_tile() failure when numresolutions=1

When trying the GDAL OpenJPEG driver against openjpeg current master HEAD,
I get failures when trying to create .jp2 files. The driver uses
opj_write_tile() and in some tests numresolutions = 1.

In openjp2/dwt.c:410, l_data_size = opj_dwt_max_resolution( tilec->resolutions,tilec->numresolutions) * (OPJ_UINT32)sizeof(OPJ_INT32);
is called and returns l_data_size = 0. Now in git opj_malloc() has a special case
for 0 to return a NULL pointer whereas previously it relied on system malloc(),
which in my case didn't return NULL.

So only test the pointer value if l_data_size != 0. This makes the GDAL
autotest suite to pass again.

Merge pull request #687 from stweil/memalign

Fix fatal crash on 64 bit Linux

Fix fatal crash on 64 bit Linux

By default, OpenJPEG uses the function memalign to allocate aligned
memory on Linux systems. That function needs malloc.h which was
missing. This results in a compiler warning:

openjpeg/src/lib/openjp2/opj_malloc.c:63:3: warning:
 implicit declaration of function ‘memalign’
 [-Wimplicit-function-declaration]

On hosts where sizeof(int) < sizeof(void *) the return value of memalign
will be truncated which results in an invalid pointer.

That caused "make test" to produce lots of segmentation faults when
running on a 64 bit Linux host.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Fix whitespace issues in opj_malloc.c

Some lines ended with spaces. Remove them.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #686 from mayeut/master

[libtiff] Add missing include statement for ssize_t
Close #681

Add missing include statement for ssize_t

Update uclouvain/openjpeg#681

Merge remote-tracking branch 'uclouvain/master'

Merge pull request #684 from stweil/typo

Fix duplicate article in comments

Fix duplicate article in comments

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge remote-tracking branch 'uclouvain/master'

Merge pull request #679 from stweil/grammar

Fix grammar in comment

Fix grammar in comment

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #665 from jwilk/spelling

Fix typos

Fix typos

Add known failures

2 more tests failing under AppVayor since nov-6 update
Update #655

Merge pull request #654 from mayeut/master

Fix undefined size jp2 box handling

Revert "Check for appveyor update"

This reverts commit c414d9c238f404980e6e8b17db652a7fb2635cdf.

Check for appveyor update

Fix missing bracket

fix appveyor build

Fix undefined size jp2 box handling

Update #653

Fix Travis cmake install

Update cmake for travis ASan build

Merge pull request #651 from stweil/fix

opj_decompress: Update error message

opj_decompress: Update error message

The png format is also supported, so add it to the message.
Remove also the unneeded blank character before \n.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Fin compiler warnings

Fix compiler warnings

convert: Fix compiler warnings

Fix this and other similar compiler warnings:

src/bin/jp2/convert.c: In function ‘tga_readheader’:
src/bin/jp2/convert.c:595:5: warning:
 dereferencing type-punned pointer will break strict-aliasing rules
 [-Wstrict-aliasing]
     cmap_len = get_ushort(*(unsigned short*)(&tga[5]));

Signed-off-by: Stefan Weil <sw@weilnetz.de>

convert: Remove unneeded type casts

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #647 from stweil/master

Fix typo in comments

Fix typo in comments

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #638 from stweil/master

Fix format strings and unneeded assignment

Merge branch 'master' into coc-qcc

Merge pull request #644 from smuehlst/opj_aligned_malloc

Avoid pointer arithmetic with (void *) pointers

Avoid pointer arithmetic with (void *) pointers.

Merge branch 'master' into opj_aligned_malloc

Conflicts:
src/lib/openjp2/opj_malloc.c

Generic aligned malloc implementation update.

Update #642

Generic aligned malloc implementation.

Used for platforms where none of posix_memalign(), memalign() and
_aligned_malloc() is available.

Generic aligned malloc implementation.

Used for platforms where none of posix_memalign(), memalign() and
_aligned_malloc() is available.

Merge pull request #641 from smuehlst/opj_hp_issue_640

Fix HP compiler warning about redeclaration of function
Fix #640
Fix #243

Fix HP compiler warning about redeclaration of function (#640)

HP compiler warns:
cc: "dwt.c", line 798: warning 562: Redeclaration of "opj_v4dwt_decode"
with a different storage class specifier: "opj_v4dwt_decode" will have
internal linkage.
cc: "t2.c", line 1341: warning 562: Redeclaration of "opj_t2_init_seg"
with a different storage class specifier: "opj_t2_init_seg" will have
internal linkage.

Fix format strings and unneeded assignment

The static code analyzer cppcheck warns about unsigned integers
which use "%d" in the format string.

It also warns about an unneeded assignment.

Fix both issues.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Merge pull request #637 from stweil/fixes

Fix repository for JPEG2000 test data

Merge branch 'master' into coc-qcc

Merge pull request #636 from uclouvain/opj_malloc-625

Update allocation functions
Fix #625
Fix #624
Fix #635

Cleanup includes

Fix _aligned_malloc usage

Fix inconsistent behavior of malloc(0)

Update #635
Update #625

Add missing checks. Fix crash on failed allocation.

Fixed crash on encoding

Update #624
Update #625

Change link for appveyor

Merge pull request #634 from smuehlst/opj_aix_issue_633

Fix  #633.

Fix OpenJPEG GitHub issue #633.

"opj_includes.h" must be included before system headers, otherwise
inconsistent definitions of configuration macros lead to build
failures on AIX.

Merge pull request #632 from mayeut/travis-reduce

travis-ci: Include add ons in matrix

travis-ci: Include add ons in matrix

Remove useless environment variables

Add ABI check for PR

Use SPDX full name & identifier for license

Add appveyor status badge & license badge

Merge branch 'master' into coc-qcc

Correct unbound variable

Merge pull request #627 from mayeut/appveyor

Add Appveyor

Remove the explicit restrict keyword

It would trigger a compiler error on xlc compiler.  Fixes #620

I do not recall exactly... I would say that the correct version should be

if (!JPWL_ASSUME)
    return false;

meaning that once JPWL_ASSUME=1 the code will pass over such errors and try to decode anyway (just paired with the other JPWL_ASSUME on line 1112).

Fixes #596

Update ci install script

Remove a possible uninitialized variable

Fixes #629

Minor style fix to remove c++ comments

minor tweaks to the code

Add appveyor.yml

Add exceptions

Find out Visual Studio version

Add exceptions

Correct exceptions

Fix an issue with parenthesis

Add exceptions

Correct script

Update kdu copy

Re-enable cloning data

update script

correct kdu copy

kdb copy

correct OPJ_SITE

run with appveyor

Add paranoid sentinels

export updated path

debug appveyor

correct warnings

export TRAVIS_OS_NAME

verbose

Use make under windows

debug appveyor

implement a portable aligned realloc

remove cygwin specifics

Debug build

Correct make script

Add OPJ_BINARY_DIR

Correct source path