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
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 6 years ago by
Tested revision 328 by tankov.vladislav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | WARNING | |||
san | SUCCESS | |||
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | WARNING | |||
san | SUCCESS | |||
Common header used in some tests: common.h
comment:3 Changed 6 years ago by
Tested revision 339 by tankov.vladislav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | WARNING | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | WARNING | |||
asan | SUCCESS | |||
clang-san | WARNING | |||
san | SUCCESS | |||
gcc62 | WARNING | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | WARNING | |||
asan | SUCCESS | |||
clang-san | WARNING | |||
san | SUCCESS | |||
Common header used in some tests: common.h
comment:4 Changed 6 years ago by
Tested revision 342 by tankov.vladislav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:5 Changed 6 years ago by
Tested revision 344 by tankov.vladislav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:6 Changed 6 years ago by
Tested revision 345 by tankov.vladislav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
run_gen_015_5x6_062x131_193x285_1x4_1x6_1x6_3x3_0_D034969E.cpp | gcc62 | FAILURE | run_gen_015_5x6_062x131_193x285_1x4_1x6_1x6_3x3_0_D034969E-gcc62.log | |
run_gen_021_8x9_4113x4861_397x815_4x4_3x4_5x5_4x4_C4F2B4F7.cpp | gcc62 | FAILURE | run_gen_021_8x9_4113x4861_397x815_4x4_3x4_5x5_4x4_C4F2B4F7-gcc62.log | |
run_gen_055_15x26_28180541x35357669_1448194617849_DF45FC8F.cpp | gcc62 | FAILURE | run_gen_055_15x26_28180541x35357669_1448194617849_DF45FC8F-gcc62.log | |
Common header used in some tests: common.h
comment:7 Changed 6 years ago by
Tested revision 347 by tankov.vladislav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:8 Changed 6 years ago by
Tested revision 348 by tankov.vladislav.
All tests passed. Good job!
comment:9 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:10 Changed 6 years ago by
Milestone: | ha3-milestone2 → ha3-deadline |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Замечания:
- Используйте
static_assert
вместоassert
в проверке полноты типа. В обоих случаях проверка выполняется на этапе компиляции, ноassert
с включеннымNDEBUG
будет скомпилирован вvoid
, т.е. проверка не будет выполнена в типичной "релизной" сборке.
- Не используйте виртуальные методы. В данном случае базовый класс это приватная деталь реализации, и никто не должен хранить
linked_ptr
по указателю на базовый класс, и виртуальный деструктор здесь не нужен.
Исправьте, пожалуйста, в ближайшие 60 часов.
comment:11 Changed 6 years ago by
Tested revision 500 by tankov.vladislav.
All tests passed. Good job!
comment:12 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:13 Changed 6 years ago by
Resolution: | → задача сдана |
---|---|
Status: | new → closed |
Решение зачтено. Бонусное задание не выполнено.
Tested revision 327 by tankov.vladislav.
Smoke tests
test
stage
result
info
log
run_linked_ptr_gen_smoke_test.cpp
gcc62
run
valgrind
run_linked_ptr_gen_smoke_test-valgrind.log
gcc62-asan
asan
clang-san
run_linked_ptr_gen_smoke_test-clang-san.log
san
run_smoke_test.cpp
gcc62
run
valgrind
exit code 1
run_smoke_test-valgrind.log
gcc62-asan
asan
exit code 1
run_smoke_test-asan.log
clang-san
run_smoke_test-clang-san.log
san
exit code 1
run_smoke_test-san.log
Common header used in some tests: common.h