#61 closed defect (fixed)
Ambiguous symbols - simple solution
Reported by: | mloskot | Owned by: | mloskot |
---|---|---|---|
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.
Attachments (1)
Change History (13)
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 , 16 years ago
Description: | modified (diff) |
---|---|
Milestone: | imported |
Reporter: | changed from | to
Resolution: | fixed |
Status: | closed → reopened |
Version: | 3.0.0 → svn-trunk |
comment:8 by , 16 years ago
Owner: | changed from | to
---|---|
Status: | reopened → new |
comment:9 by , 16 years ago
Status: | new → assigned |
---|
comment:10 by , 16 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
comment:11 by , 16 years ago
Milestone: | → 3.0.0 |
---|
Note:
See TracTickets
for help on using tickets.