New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SI-6811 Deprecate scala.util.parsing.json #2581

Merged
merged 1 commit into from May 23, 2013

Conversation

Projects
None yet
3 participants
@soc
Copy link
Member

soc commented May 22, 2013

No description provided.

@soc

This comment has been minimized.

Copy link
Member Author

soc commented May 22, 2013

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented May 22, 2013

LGTM -- assuming you got it all, we should probably track on JIRA what we'll say in the release notes and there we should recommend OSS alternatives for stuff we'll drop outright

@soc

This comment has been minimized.

Copy link
Member Author

soc commented May 23, 2013

@adriaanm One open question is what's the future of the parser combinator library in general ... do we want to keep it? Like JSON and XML, this seems to be an area where type macros could be a huge change. E. g. having a type macro which compiles the parser combinator down to a simple state machine. I think I even saw something like that on the mailing list recently ... ah here it is: https://groups.google.com/d/topic/scala-language/sXC-f88Adiw/discussion

@adriaanm

This comment has been minimized.

Copy link
Member

adriaanm commented May 23, 2013

I agree staging is a great idea for combinator parsers -- I don't think you need type macros for that: blackbox macros should do. The parsers are one of those components that I intend to ship as a separate jar for 2.11.

Blackbox macros are those macros that look just like regular method calls (type signature fully specified).

@soc

This comment has been minimized.

Copy link
Member Author

soc commented May 23, 2013

@adriaanm Alright, sounds nice. I'll add the intention to split parsers into a separate jar to SI-6811, so we don't forget it.

@paulp

This comment has been minimized.

Copy link
Contributor

paulp commented May 23, 2013

LGTM

paulp added a commit that referenced this pull request May 23, 2013

Merge pull request #2581 from soc/SI-6811-json
SI-6811 Deprecate scala.util.parsing.json

@paulp paulp merged commit 6256e22 into scala:master May 23, 2013

1 check passed

default pr-checkin-per-commit Took 66 min.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment