v75 : brl.json : JsonArray bug

Monkey Forums/Monkey Bug Reports/v75 : brl.json : JsonArray bug

AdamRedwoods(Posted 2013) [#1]
JsonArray.Get() doesn't seem to work too well.

I think it should be
	Method Get:JsonValue( index:Int )
		If index<0 Or index>=_data.Length ThrowError
		Local val:=_data[ index ]
		If val Return val
		Return JsonNull.Instance()
	End



programmer(Posted 2013) [#2]
It's already fixed: https://github.com/blitz-research/monkey/commit/9f0f9f8f38473fb815347f1f687b9f095c4b94b2


Nobuyuki(Posted 2013) [#3]
remind me to always look at the develop branch. Will replacing brl.json from this commit hose a working copy of monkey from say v74 I wonder?


marksibly(Posted 2013) [#4]
> Will replacing brl.json from this commit hose a working copy of monkey from say v74 I wonder?

Shouldn't do...


Nobuyuki(Posted 2013) [#5]
I flung another bug report up on github, seems GetBool returns Int too, not just GetFloat. Easy fix!