deprecate XMLEventReader?

[SethTisue]

it's

1. janky, proof-of-concept code from at least a decade ago
2. probably not much used, I certainly wouldn't advise it for production code
3. unrelated to the rest of the library
4. in scala.* namespace, which gives people understandably higher expectations of standard-ness and quality

maybe we should deprecate it?

if someone were to get interested in it, they could always bring it back to life as a separate library (not under scala.* this time)

[SethTisue]

I made a label https://github.com/scala/scala-xml/labels/XMLEventReader and attached to relevant open issues

[SethTisue]

(note that there are probably other XMLEventReader bugs for which tickets exist in JIRA but no one cared enough to recreate here)

[ashawley]

Sounds good for the reasons you mention. Despite all that, it is popular:

https://github.com/search?l=Scala&q=XMLEventReader&type=Code

[ashawley]

Interestingly, there is an XMLEventReader that started shipping with Java in version 6:

https://docs.oracle.com/javase/6/docs/api/javax/xml/stream/XMLEventReader.html

[SethTisue]

invited comment at https://contributors.scala-lang.org/t/shall-we-deprecate-scala-xml-pull/1636/1

[SethTisue]

Despite all that, it is popular

I'm not sure those search results qualify for being described as "popular". There is some usage out there, clearly.

Interestingly, there is an XMLEventReader that started shipping with Java in version 6

oh, interesting. well, all the more reason to deprecate ours, then.

[mghildiy]

I can take this one.

[SethTisue]

before actually doing anything, let's wait and see what the community response is to the Discourse post, if any

[SethTisue]

resounding silence on Discourse, so I think the next step is to go ahead and deprecate.

and also see if a volunteer wants to move it to a separate repo. @mghildiy you could take either or both tasks if you like

though I wouldn't suggest you do the "move to separate repo" part unless you think it's actually worth your while, I wouldn't suggest you do it only because you're looking to help out

[mghildiy]

As part of message, I should suggest a better alternative to XMLEventReader. Any good candidate for that?

[mghildiy]

..and I guess I don't need to mention "since".

[SethTisue]

I should suggest a better alternative to XMLEventReader

this seems like a safe recommendation:
https://docs.oracle.com/javase/8/docs/api/javax/xml/stream/package-summary.html

I don't know of any Scala-based alternatives.

..and I guess I don't need to mention "since".

not sure what you mean?

[SethTisue]

I don't know of any Scala-based alternatives.

I asked on the Discourse thread, just in case.