Opened 18 years ago

Last modified 15 years ago

#91 closed defect (fixed)

NodeMap.cpp: invalid conversion

Reported by: mloskot Owned by: strk@…
Priority: major Milestone:
Component: Core Version: main
Severity: Critical Keywords: imported, phpbugtracker
Cc:

Description (last modified by mloskot)

Today morning, I did cvs update and tried to do complete rebuild but I got 2 errors: 

/home/mloskot/dev/geos/_cvs/geos/source/geomgraph/NodeMap.cpp: In member function 'geos::geomgraph::Node* geos::geomgraph::NodeMap::addNode(geos::geomgraph::Node*)':
/home/mloskot/dev/geos/_cvs/geos/source/geomgraph/NodeMap.cpp:105: error: invalid conversion from 'geos::geomgraph::Node*' to 'int'
/home/mloskot/dev/geos/_cvs/geos/source/geomgraph/NodeMap.cpp:105: error: initializing argument 1 of 'geos::geomgraph::Label::Label(int)'

Change History (5)

comment:1 by mateusz@…, 18 years ago 

REPORT UPDATE

As I see, in NodeMap.cpp:105, the pointer to Node is passed, but there is no mergeLabel that expects pointer:

mloskot@dog:~/dev/geos/_cvs/geos/source$ find . -name *.h | xargs grep mergeLabel
./headers/geos/geomgraph/EdgeRing.h:    void mergeLabel(Label& deLabel);
./headers/geos/geomgraph/EdgeRing.h:    void mergeLabel(Label& deLabel, int geomIndex);
./headers/geos/geomgraph/Node.h:        virtual void mergeLabel(const Node& n);
./headers/geos/geomgraph/Node.h:        virtual void mergeLabel(const Label& label2);

comment:2 by strk@…, 18 years ago

Resolution: none → fixed 
Should be fixed in CVS. I changed signature of the function to take by ref rather then by pointer...

comment:3 by mateusz@…, 18 years ago 

I confirm this issue has been fixed. GEOS CVS compiles. Thanks!

comment:4 by mloskot, 16 years ago

Description: modified (diff)
Milestone: imported → 3.0.0
Priority: 3 → major
Reporter: changed from mateusz@… to mloskot
Version: 3.0.0 → svn-trunk

comment:5 by (none), 15 years ago

Milestone: 3.0.0

Milestone 3.0.0 deleted

Note: See TracTickets for help on using tickets.