Skip to content
This repository has been archived by the owner on Jan 25, 2023. It is now read-only.

Добавил семантику перемещения для JSONValue #3152

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

orefkov
Copy link
Contributor

@orefkov orefkov commented Dec 7, 2022

Subj

@1vanK
Copy link
Contributor

1vanK commented Dec 7, 2022

Странно, что diff показывает разницу в каждой строке. Там другой конец строки?

@1vanK
Copy link
Contributor

1vanK commented Dec 7, 2022

У тебя может локальный гит настроен, что не исправляет концы строк? https://docs.github.com/en/get-started/getting-started-with-git/configuring-git-to-handle-line-endings

@1vanK
Copy link
Contributor

1vanK commented Dec 8, 2022

Ну да, после твоих прошлых коммитов и в Manual_Core.cpp юниксовые концы строк даже на винде

@1vanK
Copy link
Contributor

1vanK commented Dec 9, 2022

Операторы/конструкторы перемещения noexcept.

Читать из union тип, который предварительно туда не был записан - нарушение strict alias rules (в си можно было, в C++ нельзя по стандарту, предлагается использовать memcpy взамен), хотя это пофигу, у нас полно такого в движке сейчас

@nobind не нужны - генератор привязок сам заигнорит это

@1vanK
Copy link
Contributor

1vanK commented Dec 9, 2022

Но генерировать биндинги все равно надо (там коммент будет, что такая-то штука не забинжена).

Еще конфликты возникли из-за того, что в репе исправлены концы строк. Легче всего пересоздать форк и оформить это в новом плуреквесте. По идее больше проблем с отправкой PR не должно возникать независимо от настроек твоего локального гита

@github-actions
Copy link

Marking this stale since there has been no activity for 30 days.
It will be closed if there is no activity for another 15 days.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants