Бычков Андрей

andrey commented on pull request SHS/pip#44

2020-10-02 10:59:44 +03:00

у тебя на 3 строки 5 строк описания дефайна, мне это кажется нелогичным, не говоря от том что макросы нужно использовать только по необходимости, коей здесь не видно

andrey commented on pull request SHS/pip#44

2020-10-02 10:58:27 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

не читаемый код получается, и к тому же кода больше чем если ты эту строку вставишь в функцию

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

надо бы заменить на memcmp

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

надо бы заменить на memcmp

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

а этот макрос теперь нужен вообще? он же в 1м месте только

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

этот макрос тоже теперь не нужен?

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

а почему этот оператор не inline, а оператор << inline?

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

Пихать в static_assert std::is_trivially_copyable::value мне кажется лишним, другой туда не попадёт, наглядней будет static_assert(false) или static_assert(0). Ещё предлагаю в сообщение запихнуть имя типа, а то не понятно на что он сработал.

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

правильнее назвать функцию assign

andrey commented on pull request SHS/pip#44

2020-10-02 10:36:10 +03:00

remove __PICONTAINERS_SIMPLE_TYPE__

тут правильнее во всех функциях использовать final вместо override