Skip to content
This repository has been archived by the owner on Jul 13, 2021. It is now read-only.

HW by Kononov Kirill #3

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

HW by Kononov Kirill #3

wants to merge 3 commits into from

Conversation

obizzz
Copy link

@obizzz obizzz commented Nov 14, 2019

Надеюсь, все гуд

@AufarZakiev
Copy link
Collaborator

Не прошло проверку форматирования. Проверьте все .cpp и .h файлы clang-format-ом

@AufarZakiev AufarZakiev removed their request for review November 14, 2019 20:31
@obizzz
Copy link
Author

obizzz commented Nov 14, 2019

Не прошло проверку форматирования. Проверьте все .cpp и .h файлы clang-format-ом

Да, я один файл пропустил при форматировании, сейчас все ок

@AufarZakiev
Copy link
Collaborator

Request review?

@AufarZakiev AufarZakiev self-requested a review November 14, 2019 20:37
++p;
}
while (*p++ = *from++)
;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что-то некрасиво получилось с пустой строкой.
Может, лучше перенести ее на строку выше или добавить пустое тело цикла { } ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Это было автоматическое форматирование, исправлю сейчас

Copy link
Collaborator

@AufarZakiev AufarZakiev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Гуд!

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.

2 participants