Opened 18 years ago
Last modified 15 years ago
#61 closed defect
Ambiguous symbols - simple solution — at Version 7
Reported by: | mloskot | Owned by: | |
---|---|---|---|
Priority: | 1 | Milestone: | |
Component: | Core | Version: | main |
Severity: | Annoyance | Keywords: | imported, phpbugtracker |
Cc: |
Description (last modified by )
This is a very simple solution for ambiguous symbols introduced in report:
#57 - Class naming changes proposal
As we talked today with strk, the #57 requires wider discussion, so for now I suggest to use simple solution - qualified name like:
using namespace geos; //... geom::Polygon
Example of error send by VC++ 8.0:
error C2872: 'Polygon' : ambiguous symbol geossourceoperationpredicateRectangleIntersects.cpp 131
Simple fix is in the patch attached to this report.
Change History (8)
comment:2 by , 18 years ago
I'm ok with using geom::Polygon, if you feel like patching all those files
comment:3 by , 18 years ago
OK, I'll fix it during the weekend. You can assign me to this report, to not to forget about it :-)
comment:5 by , 18 years ago
Fixed in patch attached. This patch applies fully qualified name to ambiugous classes, so Polygon is changed to geos::geom::Polygon or geom::Polygon (when using namespace geos; directive is used). Please, close this report after patch will be applied.
by , 18 years ago
Attachment: | operation-predicate-full-path-of-polygon-mloskot-20060319.3.patch added |
---|
Generated with cvs diff -U2
comment:6 by , 18 years ago
Resolution: | none → fixed |
---|
comment:7 by , 17 years ago
Description: | modified (diff) |
---|---|
Milestone: | imported |
Reporter: | changed from | to
Resolution: | fixed |
Status: | closed → reopened |
Version: | 3.0.0 → svn-trunk |
Note:
See TracTickets
for help on using tickets.