Opened 6 years ago

Closed 6 years ago

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

Проверка ДЗ №3

Reported by: tankov.vladislav Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha3-deadline
Component: HA#3 linked_ptr Version:
Keywords: Cc:

Description


Change History (13)

comment:1 Changed 6 years ago by cpptester

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

Tested revision 327 by tankov.vladislav.

Smoke tests

test

stage

result

info

log

run_linked_ptr_gen_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

SUCCESS

run_linked_ptr_gen_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

SUCCESS

clang-san

WARNING

run_linked_ptr_gen_smoke_test-clang-san.log

san

SUCCESS

run_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

FAILURE

exit code 1

run_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

FAILURE

exit code 1

run_smoke_test-asan.log

clang-san

WARNING

run_smoke_test-clang-san.log

san

FAILURE

exit code 1

run_smoke_test-san.log

Common header used in some tests: common.h

comment:2 Changed 6 years ago by cpptester

Tested revision 328 by tankov.vladislav.

Smoke tests

test

stage

result

info

log

run_linked_ptr_gen_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

SUCCESS

run_linked_ptr_gen_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

SUCCESS

clang-san

WARNING

run_linked_ptr_gen_smoke_test-clang-san.log

san

SUCCESS

run_smoke_test.cpp

gcc62

SUCCESS

run

SUCCESS

valgrind

SUCCESS

run_smoke_test-valgrind.log

gcc62-asan

SUCCESS

asan

SUCCESS

clang-san

WARNING

run_smoke_test-clang-san.log

san

SUCCESS

Common header used in some tests: common.h

comment:3 Changed 6 years ago by cpptester

Tested revision 339 by tankov.vladislav.

Smoke tests

test

stage

result

info

log

run_linked_ptr_gen_smoke_test.cpp

gcc62

WARNING

run_linked_ptr_gen_smoke_test-gcc62.log

run

SUCCESS

valgrind

SUCCESS

run_linked_ptr_gen_smoke_test-valgrind.log

gcc62-asan

WARNING

run_linked_ptr_gen_smoke_test-gcc62-asan.log

asan

SUCCESS

clang-san

WARNING

run_linked_ptr_gen_smoke_test-clang-san.log

san

SUCCESS

run_smoke_test.cpp

gcc62

WARNING

run_smoke_test-gcc62.log

run

SUCCESS

valgrind

SUCCESS

run_smoke_test-valgrind.log

gcc62-asan

WARNING

run_smoke_test-gcc62-asan.log

asan

SUCCESS

clang-san

WARNING

run_smoke_test-clang-san.log

san

SUCCESS

Common header used in some tests: common.h

comment:7 Changed 6 years ago by cpptester

Tested revision 347 by tankov.vladislav.

Main tests

test

stage

result

info

log

ncomp_include_iostream.cpp

gcc62

FAILURE

Common header used in some tests: common.h

comment:8 Changed 6 years ago by cpptester

Tested revision 348 by tankov.vladislav.

All tests passed. Good job!

comment:9 Changed 6 years ago by tankov.vladislav

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

comment:10 Changed 6 years ago by Vladimir Rutsky

Milestone: ha3-milestone2ha3-deadline
Type: ожидается проверкаожидаются исправления

Замечания:

  1. Используйте static_assert вместо assert в проверке полноты типа. В обоих случаях проверка выполняется на этапе компиляции, но assert с включенным NDEBUG будет скомпилирован в void, т.е. проверка не будет выполнена в типичной "релизной" сборке.
  1. Не используйте виртуальные методы. В данном случае базовый класс это приватная деталь реализации, и никто не должен хранить linked_ptr по указателю на базовый класс, и виртуальный деструктор здесь не нужен.

Исправьте, пожалуйста, в ближайшие 60 часов.

comment:11 Changed 6 years ago by cpptester

Tested revision 500 by tankov.vladislav.

All tests passed. Good job!

comment:12 Changed 6 years ago by tankov.vladislav

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

comment:13 Changed 6 years ago by Vladimir Rutsky

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

Решение зачтено. Бонусное задание не выполнено.

Note: See TracTickets for help on using tickets.