Doesn't correctly parse numbers immediately before a brace.

[rickh57]

I'm using the covid-193.p.rapidapi.com. It returns JSON like this:

{"get":"statistics","parameters":{"country":"India"},"errors":[],"results":1,"response":[{"continent":"Asia","country":"India","population":1387975211,"cases":{"new":null,"active":164278,"critical":8944,"recovered":10448406,"1M_pop":"7757","total":10767206},"deaths":{"new":null,"1M_pop":"111","total":154522},"tests":{"1M_pop":"142475","total":197752057},"day":"2021-02-02","time":"2021-02-02T14:45:06+00:00"}]}

After parsing with JsonItem and dumping, I see this:

{
  "get" : "statistics",
  "parameters" : {
    "country" : "USA"
  },
  "errors" : ,
  "results" : 1,
  "response" : [
    {
      "continent" : "North-America",
      "country" : "USA",
      "population" : 332145637,
      "cases" : {
        "new" : "+1597",
        "active" : 9825868,
        "critical" : 23860,
        "recovered" : 16632858,
        "1M_pop" : "81028",
        "total" : 
      },
      "deaths" : {
        "new" : "+33",
        "1M_pop" : "1368",
        "total" : 
      },
      "tests" : {
        "1M_pop" : "945964",
        "total" : 
      },
      "day" : "2021-02-02",
      "time" : "2021-02-02T14:45:06+00:00"
    }
  ]
}

Numbers directly followed by ,, such as "recovered":10448406, are parsed OK. However, "total":154522} isn't parsed -- with an empty string being captured.

[StringEpsilon]

Thank you for the report. Should be fixed in the latest commit. I tested it with your file. Let me know whether or not it works for you.

[rickh57]

I commented on the commit, but I'll comment here, too. It's working fine now with the latest commit. I glanced at the code and was considering trying to fix it myself, but you certainly beat me to it.

[StringEpsilon]

You're brave. I wrote this library with no regard for maintainability or even comprehensibility.

[rickh57]

Hah -- I've been an engineer since 1988 -- so have seen much uglier code than yours. I'm sure that I could have figured it out -- eventually. :)

…

On Tue, Feb 2, 2021 at 9:44 AM StringEpsilon ***@***.***> wrote: You're brave. I wrote this library with no regard for maintainability or even comprehensibility. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#10 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEIYQZFFNRGSZ3H2PPF4VTLS5A2XFANCNFSM4W6ZQMKQ> .

-- ------------------------------------------------ Rick Hale (rickh57@gmail.com)