On Thu, Nov 29, 2012 at 7:29 PM, Justin Deoliveira <[email protected]>wrote:

> Hi all,
>
> Recently myself and a colleague of mine (Rob Marianski) worked on some
> improved parsing for KML. Including support for parsing KML 2.2 along with
> some of its advanced "custom data" features, ie "SchemaData" and
> "ExtendedData".
>
> The work is mostly constrained to the kml bindings which afaik don't see
> much use but there were a few core changes required, including adding a new
> PullParser to the xsd-core module. Written against the jaxp XMLStreamReader
> interface it should be a good replacement for STreamingParser. While this
> was not required for the KML work it seemed worth it to come up with a
> better implementation for stream parsing.
>
> Anyways, the changes have been pushed to master but i was hoping to
> backport to 8.x. So i opened a pull request for review:
>
>   https://github.com/geotools/geotools/pull/67
>

Hmm... clean, well tested, mostly additions to the API, I agree the KML
does not seem to be used
much anyways and this could kick its usage up.
+1 on the backport provided PullParser API is reasonably meant to be stable
over time (not saying
not changing ever, but not changing next week)

Wondering about StreamingParser... should it be deprecated?
More in general, what's the relationship between it and PullParser?
One replaces the other, they have different features and thus different
intended
use cases?

Cheers
Andrea


-- 
==
Our support, Your Success! Visit http://opensdi.geo-solutions.it for more
information.
==

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39  339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

-------------------------------------------------------
------------------------------------------------------------------------------
Keep yourself connected to Go Parallel: 
TUNE You got it built. Now make it sing. Tune shows you how.
http://goparallel.sourceforge.net
_______________________________________________
GeoTools-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to