Skip to content
This repository was archived by the owner on Jul 19, 2022. It is now read-only.
This repository was archived by the owner on Jul 19, 2022. It is now read-only.

\u unicode values are not parsed correctly #47

@gcamp

Description

@gcamp

Seems like \u values are still not parsed correctly.

Ex : \u00e7 (can be found in https://open.tan.fr/ewp/tempsattente.json/COMB2)

I'm not entirely confortable with specs, but it seems that a big part of it is not supported. For example, the current code doesn't support surrogate pair.

We can also compare with what other json parser are doing and it is quite different :
https://github.com/miloyip/rapidjson/blob/master/include/rapidjson/reader.h#L696
https://github.com/open-source-parsers/jsoncpp/blob/master/src/lib_json/json_reader.cpp#L682

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions