Unify prototypes of WKTReader and WKBReader constructor

[mloskot]

Constructor of both WKTReader and WKBReader accept GeometryFactory, however both do it in different way:

• WKBReader​ accepts reference to const GeometryFactory object
• WKTReader​ accepts pointer

This is inconsistent and requires extra work on client side. For example, if one wants to write simple function or class template that can be specialised by any of the two reader types it becomes a challenge.

I would suggest to change ctor of WKTReader to accept reference instead of pointer.

[strk]

+1

Btw, wouldn't you call these "troubles" ? :)

[strk]

Done in r3770

[Sandro Santilli <strk@…>]

In ddab738/git:

Deprecate WKTReader constructor taking GeometryFactory by pointer

Add constructor taking it by reference, for consistency with WKBReader
Closes #310

git-svn-id: ​http://svn.osgeo.org/geos/trunk@3770 5242fede-7e19-0410-aef8-94bd7d2200fb

[Sandro Santilli <strk@…>]

In ddab738/git:

Deprecate WKTReader constructor taking GeometryFactory by pointer

Add constructor taking it by reference, for consistency with WKBReader
Closes #310

git-svn-id: ​http://svn.osgeo.org/geos/trunk@3770 5242fede-7e19-0410-aef8-94bd7d2200fb

[Sandro Santilli <strk@…>]

In ddab738/git:

Deprecate WKTReader constructor taking GeometryFactory by pointer

Add constructor taking it by reference, for consistency with WKBReader
Closes #310

git-svn-id: ​http://svn.osgeo.org/geos/trunk@3770 5242fede-7e19-0410-aef8-94bd7d2200fb