Opened 5 years ago

Closed 5 years ago

#12 closed ожидается проверка (задача сдана)

WW #1 golovin.valeriy

Reported by: golovin.valeriy Owned by: ldvsoft
Component: WW_make Version:
Keywords: Cc:

Description


Change History (5)

comment:1 Changed 5 years ago by Egor Suvorov

Component: HW #1 (BMP) → WW_make
Owner: changed from Evgeny Linsky to Дмитрий Лапшин (lapshin)
Status: new → assigned

Пожалуйста, указывайте корректные поля при создании тикета.

comment:2 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Owner: changed from Дмитрий Лапшин (lapshin) to golovin.valeriy
Summary: WW #1 → WW #1 golovin.valeriy
Type: ожидается проверка → ожидаются исправления

Привет!

Выглядит всё хорошо, но есть недостатки:

  1. Стиль кода местами страдает, причеши: мы всегда обособляем инфиксные (которые между двумя аргументами) операторы пробелами: a / b, и пустые строки в внезапных местах (например первой строчкой) не помогают.
  2. Также попрошу придерживаться единого, одинакого, отступа. Мне не важно, какой он именно, пока он адекватный. Например, многие любят 4 пробела, или символ табуляции.
  3. Заголовочные файлы хороши, но лучше пропиши-таки имена аргументов функций.

comment:3 Changed 5 years ago by golovin.valeriy

Type: ожидаются исправления → ожидается проверка
Version: 1.0

comment:4 Changed 5 years ago by golovin.valeriy

Owner: changed from golovin.valeriy to ldvsoft
Summary: WW #1 golovin.valeriy → WW #1

comment:5 Changed 5 years ago by Дмитрий Лапшин (lapshin)

Resolution: → задача сдана
Status: assigned → closed
Summary: WW #1 → WW #1 golovin.valeriy

На меня плохо назначил ж! Лишний пробел что ли?

Но ОК.

Note: See TracTickets for help on using tickets.