Skip to content

Fix violation of rule of 5 in JsonObject #4

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 7 commits into from
May 19, 2025

Conversation

TymianekPL
Copy link
Collaborator

object.hpp and object.cpp

@TymianekPL
Copy link
Collaborator Author

I'll fix operator[] issues in clang in another pr, merging

@TymianekPL TymianekPL merged commit 1770090 into tymi/json-object May 19, 2025
3 of 6 checks passed
@TymianekPL TymianekPL deleted the tymi/fix-JsonObject-5 branch May 19, 2025 06:48
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