Skip to content
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

Необходимо ввести стандартизацию #39

Closed
vadim-su opened this issue Feb 11, 2014 · 15 comments
Closed

Необходимо ввести стандартизацию #39

vadim-su opened this issue Feb 11, 2014 · 15 comments

Comments

@vadim-su
Copy link

Думаю будет нелишнем написать принцип оформления кода и его комментирования.
Нужность этого действия неоспорима и в обсуждении не нуждается, а вот правила стандарта предлагаю обсудить здесь.

@mikhail4021
Copy link
Contributor

Думаю будет нелишнем установить spellchecker. Нужность этого действия неоспорима и в обсуждении не нуждается.

@vadim-su
Copy link
Author

Ок, а по делу будут комментарии, а то я уж обрадовался, что есть предложения.

@mikhail4021
Copy link
Contributor

А по делу, код и так неплохо написан для OpenSource.

@orignal
Copy link
Contributor

orignal commented Feb 11, 2014

deterok, а есть какие то конкретные претензии?
По существу вопроса:

  1. писать имена и комментарии на английском языке.
  2. если какая то функциональность присуствует в std то использовать ее, а не boost

@chertov
Copy link
Contributor

chertov commented Feb 12, 2014

deterok, не думаю что так уж важно
стоит посмотреть стиль который использует orignal и придерживаться такого же т.к. большая часть кода в этом стиле)
лично я привык ставить больше пробелов в условиях, между параметрами в методах и т.д, но кода я написал совсем мало... не так и трудно соблюдать общую стилистику автора за которым 99.99% всего кода... был бы код "размазан" сильно по нескольким кодерам с разным стилем, то пришлось бы договариваться

@mlt
Copy link
Contributor

mlt commented Jul 22, 2015

Recent commit 4412dd1 was a real PITA. Can we have at least a config for astyle, beautifier, uncrustify or something so that one can git filter local changes before rebasing?

P.S. Could someone with write access translate the title?

@orignal
Copy link
Contributor

orignal commented Jul 22, 2015

Михаил, я больше этим проектом не занимаюсь.

@vadim-su
Copy link
Author

А что так?

@orignal
Copy link
Contributor

orignal commented Jul 23, 2015

По причине бесперспективности.

@vadim-su
Copy link
Author

Ok

@orignal
Copy link
Contributor

orignal commented Oct 20, 2015

Ну что, господа, имею вам сказать, что большинство у меня большинство старых и новых багов пофиксено, проблемы с утечками памяти решены, EdDSA тоже заработал.

@vadim-su
Copy link
Author

Вернулся?

@orignal
Copy link
Contributor

orignal commented Oct 21, 2015

Как видишь. Но поскольку здесь я оказался лишним пришлось переползти на bitbucket

@iShift
Copy link
Contributor

iShift commented Oct 21, 2015

@orignal попили Tox тогда лучше там и комюнити по больше и проект более масштабный =)

@orignal
Copy link
Contributor

orignal commented Oct 21, 2015

А мне что с него?
Но вообще наверное гляну.

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

No branches or pull requests

7 participants