Opened 5 years ago

Closed 5 years ago

#942 closed defect (fixed)

GeometryFactory.cpp:750:1: warning: control reaches end of non-void function on windows cmake

Reported by: robe Owned by: robe
Priority: minor Milestone: 3.8.0
Component: Buildbots Version: main
Severity: Annoyance Keywords:
Cc:

Description

I'm seeing this warning on winnie's 32-bit and 64-bit runs. Not sure why I am not seeing it on the other bots.

E:/jenkins/geos/branches/3.8cmake/src/geom/GeometryFactory.cpp: In member function 'geos::geom::Geometry* geos::geom::GeometryFactory::buildGeometry(const std::vector<geos::geom::Geometry*>&) const':
E:/jenkins/geos/branches/3.8cmake/src/geom/GeometryFactory.cpp:750:1: warning: control reaches end of non-void function [-Wreturn-type]
 }

This is building against GCC 4.8.3 and GCC 4.8.1.

I presume because it's just doing an assert rather than returning something so warning seems valid to me.

Change History (3)

comment:1 by robe, 5 years ago

I checked to see why I'm not getting the notice in 3.7 run, it appears in 3.8 we took ou the - return geom0->clone(); that is the catch all when nothing returns.

Aside from that the buildGeometry seems to have been considerably rewritten.

comment:2 by robe, 5 years ago

Reopened the pull request at - https://github.com/libgeos/geos/pull/129 that triggered this. Was a result of fixing #929

comment:3 by Algunenano, 5 years ago

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.