U++ framework: U++ Core » ParseJSON now throws exception

Home » U++ Library support » U++ Core » ParseJSON now throws exception

Show: Today's Messages :: Polls :: Message Navigator
E-mail to friend

ParseJSON now throws exception [message #49135]

Sun, 24 December 2017 21:48

Mindtraveller
Messages: 917
Registered: August 2007
Location: Russia, Moscow rgn.

Experienced Contributor

If non-json data is met, old version of ParseJSON just returned error Value. The current version of ParseJSON looks like thrown an exception. This is not very good because it is different behavior which conflicts with old codebase. So I propose considering it a bug.

Report message to a moderator

Re: ParseJSON now throws exception [message #49136 is a reply to message #49135]

Mon, 25 December 2017 07:16

mirek
Messages: 14290
Registered: November 2005

Ultimate Member

Mindtraveller wrote on Sun, 24 December 2017 21:48

If non-json data is met, old version of ParseJSON just returned error Value. The current version of ParseJSON looks like thrown an exception. This is not very good because it is different behavior which conflicts with old codebase. So I propose considering it a bug.

There was no change.

However, there were always two overloads:

Value ParseJSON(CParser& p);
Value ParseJSON(const char *s);

First one throws, second one returns Void.

Report message to a moderator