Hi Bob,
Could you please add an "annotated tag" to your git repository each time you make a fastJson release with a version number? This will make it show up in the "Releases" list on GitHub and will make it much easier to check what the latest release is!
The Creating Releases in the GitHub documentation has a step-by-step guide to adding a release to your GitHub repository.
Thanks!
"new" JSON library fastJson
Moderators: FourthWorld, heatherlaine, Klaus, kevinmiller, robinmiller
Re: "new" JSON library fastJson
LiveCode Open Source Team — @PeterTBBrett — peter.brett@livecode.com
Re: "new" JSON library fastJson
I'll start creating versions to make it easier. Thanks for the tip.
The solution is really easy to the performance issue. I simply made a case in the parsing switch statement looking for "" and simply returning it.
I have no idea what's happening but I don't think it's a bug in Livecode. I'm suspecting it's something to do with a UTF function getting an empty string where it's expecting no null for strings. Not sure. Didn't take the time to dive into it as I spent so much time trying to figure out what was going on and had to get back to paying projects
I did put some time into the isNumericalArray() and I'd love for some folks to push it and see if I finally got it right.
Thanks again to everyone for feedback and assistance in debugging this!
Bob
The solution is really easy to the performance issue. I simply made a case in the parsing switch statement looking for "" and simply returning it.
I have no idea what's happening but I don't think it's a bug in Livecode. I'm suspecting it's something to do with a UTF function getting an empty string where it's expecting no null for strings. Not sure. Didn't take the time to dive into it as I spent so much time trying to figure out what was going on and had to get back to paying projects
I did put some time into the isNumericalArray() and I'd love for some folks to push it and see if I finally got it right.
Thanks again to everyone for feedback and assistance in debugging this!
Bob
-
- VIP Livecode Opensource Backer
- Posts: 3581
- Joined: Mon Jan 22, 2007 7:36 am
- Location: Berkeley, CA, US
- Contact:
Re: "new" JSON library fastJson
OK - I submitted a pull request for you.
The isNumericalArray change didn't make it into the library stack, just the .lc file.
And the library is scriptified now, so github can handle the diffs.
Otherwise looks good, and the timing is as good as before.
The isNumericalArray change didn't make it into the library stack, just the .lc file.
And the library is scriptified now, so github can handle the diffs.
Otherwise looks good, and the timing is as good as before.
PowerDebug http://powerdebug.ahsoftware.net
PowerTools http://www.ahsoftware.net/PowerTools/PowerTools.irev
PowerTools http://www.ahsoftware.net/PowerTools/PowerTools.irev