|
The reason I initially filed this ticket is because "fromjson" (the JSON parser) would use the logging framework when compiled with debug printouts, and at the time that would cause a crash when "fromjson" was called from a static context. I tested it with a recent version, and it now works, possibly because logging can now be used safely in a static context.
Given that, doing this may not have any value, and this work may be done as part of a larger refactor, such as SERVER-11919 or a similar ticket that has the potential to make "fromjson" no longer static context callable (although it's conceptually simple, so I see no reason why it shouldn't always be safe to call from a static context).
My recommendation, then, is to mark this as "Works As Designed" for now, since it's low value cleanup and will only come up again if someone is doing something to make the parser unsafe to call from a static context. matt.kangas@10gen.com and milkie, what do you think?
|