This page is no longer maintained — Please continue to the home page at www.scala-lang.org

Fix for #3284

2 replies
Derek Chen-Becker
Joined: 2008-12-16,
User offline. Last seen 42 years 45 weeks ago.

I have a fix in place for #3284 that should not break the build this
time (again, sorry!). Is it OK to check this in or should I hold off?
Also, this change modifies the JSON parsing api slightly (parse is
deprecated in favor of parseRaw due to some underlying architecture
changes), so I was wondering if this is something I should submit to
review board before proceeding. I've attached the diff in case anyone
wants to take a look at it.

Derek

adriaanm
Joined: 2010-02-08,
User offline. Last seen 31 weeks 4 days ago.
Re: Fix for #3284
I had a quick look and it looks ok to me.
Thanks!adriaan

On Fri, Apr 16, 2010 at 7:27 PM, Derek Chen-Becker <java@chen-becker.org> wrote:
I have a fix in place for #3284 that should not break the build this
time (again, sorry!). Is it OK to check this in or should I hold off?
Also, this change modifies the JSON parsing api slightly (parse is
deprecated in favor of parseRaw due to some underlying architecture
changes), so I was wondering if this is something I should submit to
review board before proceeding. I've attached the diff in case anyone
wants to take a look at it.

Derek

Derek Chen-Becker
Joined: 2008-12-16,
User offline. Last seen 42 years 45 weeks ago.
Fix for #3284

I've gone ahead and commited the fix for #3284 (JSON Parsing) with
Martin's OK (per Antonio). I have not done the svnmerge because I'm a
bit wary of doing that myself. Could someone please look at r22170 and
merge that into the 2.8 branch?

Thanks,

Derek

Copyright © 2012 École Polytechnique Fédérale de Lausanne (EPFL), Lausanne, Switzerland