Opened 6 years ago

Closed 5 years ago

#904 closed defect (fixed)

warning C4189 (appVeyor)

Reported by: cvvergara Owned by: geos-devel@…
Priority: minor Milestone: 3.8.0
Component: Unit Tests Version: 3.6.2
Severity: Annoyance Keywords:
Cc:

Description

On AppVeyor:

bool d = delete_obj();
assert(d && "delete failed with SEH disabled: runtime bug?");

c:\projects\geos\tests\unit\tut\tut.hpp(238): warning C4189: 'd': local variable is initialized but not referenced (compiling source file C:\projects\geos\tests\unit\capi\GEOSVoronoiDiagramTest.cpp) [C:\projects\geos\tests\unit\test_geos_unit.vcxproj]

Which means that the assertions are not processed.

But not such warning occurs when processing on Travis, which means that the assertions are being processed.

Possible solutions: 1) where needed:

::geos::ignore_unused_variable_warning(d);

2) on appveyor.yml

configuration: Debug

Change History (2)

comment:1 by robe, 6 years ago

Milestone: 3.6.33.8.0

comment:2 by Daniel Baston <dbaston@…>, 5 years ago

Resolution: fixed
Status: newclosed

In 2cd5f5b/git:

Attempt to quiet TUT warnings in both MSVC, gcc

Fixes #904

Note: See TracTickets for help on using tickets.