Opened 4 years ago

Closed 4 years ago

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

HA6: bind

Reported by: goncharov.artem Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha6-deadline
Component: HA#6 bind Version: 1.0
Keywords: Cc:

Description

Проверьте, пожалуйста.

Change History (13)

comment:1 Changed 4 years ago by cpptester

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

Tested revision 780 by goncharov.artem.

Smoke tests

test

stage

result

info

log

run_bind_smoke_test.cpp

FAILURE

run_bind_smoke_test-gcc62.log

Common header used in some tests: common.h

comment:2 Changed 4 years ago by cpptester

Tested revision 782 by goncharov.artem.

Smoke tests

test

stage

result

info

log

run_bind_smoke_test.cpp

WARNING

run_bind_smoke_test-gcc62.log

SUCCESS
SUCCESS

run_bind_smoke_test-valgrind.log

WARNING

run_bind_smoke_test-gcc62-asan.log

SUCCESS
WARNING

run_bind_smoke_test-clang-san.log

SUCCESS

Common header used in some tests: common.h

comment:3 Changed 4 years ago by cpptester

Tested revision 783 by goncharov.artem.

Smoke tests

test

stage

result

info

log

run_bind_smoke_test.cpp

WARNING

run_bind_smoke_test-gcc62.log

SUCCESS
SUCCESS

run_bind_smoke_test-valgrind.log

WARNING

run_bind_smoke_test-gcc62-asan.log

SUCCESS
SUCCESS
SUCCESS

Common header used in some tests: common.h

comment:5 Changed 4 years ago by cpptester

Tested revision 785 by goncharov.artem.

Main tests

test

stage

result

info

log

ncomp_bind_missing_arg.cpp

FAILURE

ncomp_bind_missing_arg2.cpp

FAILURE

Common header used in some tests: common.h

comment:6 Changed 4 years ago by cpptester

Tested revision 786 by goncharov.artem.

All tests passed. Good job!

comment:7 Changed 4 years ago by goncharov.artem

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

comment:8 Changed 4 years ago by Vladimir Rutsky

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

Замечания:

  1. Вместо std::int32_t используйте size_t для индексов.
  1. Сейчас плейсхолдеры имеют внешнюю линковку --- ваше решение не слинкуется, если несколько единиц трансляций будут использовать bind.hpp:
 Placeholder<0> _1;
 Placeholder<1> _2;

Сделайте их статическими (либо используя static, либо const).

  1. В плейсхолдерах не требуется статическое поле value --- вы его не используете, а производите выбор по шаблонному параметру.

Исправьте, пожалуйста.

comment:10 Changed 4 years ago by cpptester

Tested revision 796 by goncharov.artem.

All tests passed. Good job!

comment:11 Changed 4 years ago by goncharov.artem

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

comment:12 Changed 4 years ago by goncharov.artem

Исправил. Посмотрите, пожалуйста.

comment:13 Changed 4 years ago by Vladimir Rutsky

Resolution: задача сдана
Status: newclosed

Решение зачтено.

Note: See TracTickets for help on using tickets.