Skip to content

Fix ambiguous operator[] #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 19, 2025
Merged

Conversation

TymianekPL
Copy link
Collaborator

Fixes #5

@TymianekPL TymianekPL merged commit 7b082ca into tymi/json-object May 19, 2025
6 checks passed
@TymianekPL TymianekPL deleted the tymi/fix-ambiuous-clang branch May 19, 2025 15:43
TymianekPL added a commit that referenced this pull request May 23, 2025
* cppjson::JsonObject primitive, tagged union

Signed-off-by: TymianekPL <[email protected]>

* std::formatter specialisation for json objects

Signed-off-by: TymianekPL <[email protected]>

* make formatter specialisation sexy ⚡

Signed-off-by: TymianekPL <[email protected]>

* sync with pr3: Changes C++23 => C++20 in .clang-format

Signed-off-by: TymianekPL <[email protected]>

* Add cppjson::Object

Signed-off-by: TymianekPL <[email protected]>

* Fix clang-format

Signed-off-by: TymianekPL <[email protected]>

* cppjson::Object

Signed-off-by: TymianekPL <[email protected]>

* mark conversions as explicitly implicit

Signed-off-by: TymianekPL <[email protected]>

* formatter specialisations for proxies

Signed-off-by: TymianekPL <[email protected]>

* Sub objects

Signed-off-by: TymianekPL <[email protected]>

* Fix violation of rule of 5 in JsonObject (#4)

* Add declarations for JsonObject's special functions

* Define copy/move constructors

* Define copy/move assignment operators

* Implicitly create bytes

* Implicitly create bytes in exchange()

* Fix includes lol (utility, cstring)

* Destroy() objects

* Fix ambiguous operator[] (#6)

* Fix ambiguous operator[]

* template typo lol

* Call Destroy() in type erase

Signed-off-by: TymianekPL <[email protected]>

* Ensure safe cleanup of arrays

Signed-off-by: TymianekPL <[email protected]>

* Arrays

Signed-off-by: TymianekPL <[email protected]>

* add emplace

* Split into multiple files & add finish arrays

Signed-off-by: TymianekPL <[email protected]>

---------

Signed-off-by: TymianekPL <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant