#255 closed ожидаются исправления (задача сдана)
Домашнее задание №3 linked_ptr
Reported by: | Andrey Chausov | Owned by: | rutsky,grabovoy.philipp |
---|---|---|---|
Priority: | проверка | Milestone: | ha3-deadline |
Component: | HA#3 linked_ptr | Version: | 1.0 |
Keywords: | Cc: | dxahtepb@… |
Description
Change History (20)
comment:1 Changed 6 years ago by
Cc: | dxahtepb@… added |
---|---|
Version: | → 1.0 |
comment:2 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:3 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:4 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 334 by chausov.andrej.
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 | |||
Common header used in some tests: common.h
comment:5 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:6 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 335 by chausov.andrej.
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:7 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:8 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 337 by chausov.andrej.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:9 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:10 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 341 by chausov.andrej.
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:11 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:12 Changed 6 years ago by
Tested revision 343 by chausov.andrej.
All tests passed. Good job!
comment:13 Changed 6 years ago by
Milestone: | ha3-milestone2 → ha3-deadline |
---|---|
Type: | ожидается проверка → ожидаются исправления |
Замечания:
bool is_convertable(U * ptr)
не во всех путях исполнения возвращает значение.
assert(is_convertable(data));
проверяется во время выполнения, хотя это можно сделать во время компиляции. Плюс, с включенным макросомNDEBUG
,assert
будет скомпилирован какvoid
, и проверка не выполнится.
- Проверка
is_convertable
излишня: после проверки вы делаете присвоениеthis->data = data;
, которое скомпилируется только тогда, когда типы неявно приводимы.
Исправьте, пожалуйста, в ближайшие 60 часов.
comment:14 Changed 6 years ago by
Tested revision 498 by chausov.andrej.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
FAILURE | ||||
FAILURE | ||||
Common header used in some tests: common.h
comment:15 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:16 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 499 by chausov.andrej.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
FAILURE | ||||
FAILURE | ||||
Common header used in some tests: common.h
comment:17 Changed 6 years ago by
Tested revision 513 by chausov.andrej.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
WARNING | ||||
SUCCESS | ||||
SUCCESS | ||||
WARNING | ||||
SUCCESS | ||||
WARNING | ||||
SUCCESS | ||||
WARNING | ||||
SUCCESS | ||||
SUCCESS | ||||
WARNING | ||||
SUCCESS | ||||
WARNING | ||||
SUCCESS | ||||
Common header used in some tests: common.h
comment:18 Changed 6 years ago by
Tested revision 514 by chausov.andrej.
All tests passed. Good job!
comment:19 Changed 6 years ago by
Resolution: | → задача сдана |
---|---|
Status: | new → closed |
is_convertable()
теперь не используется и её стоит удалить.
Решение зачтено. Бонусное задание не выполнено.
comment:20 Changed 6 years ago by
Tested revision 518 by chausov.andrej.
All tests passed. Good job!
Tested revision 333 by chausov.andrej.
Main tests
test
stage
result
info
log
comp_all_sufficient.cpp
gcc62
comp_all_sufficient-gcc62.log
comp_assign.cpp
gcc62
comp_assign-gcc62.log
comp_assign_const.cpp
gcc62
comp_assign_const-gcc62.log
comp_assign_derived.cpp
gcc62
comp_assign_derived-gcc62.log
comp_assign_derived_const.cpp
gcc62
comp_assign_derived_const-gcc62.log
comp_assign_result.cpp
gcc62
comp_assign_result-gcc62.log
comp_assign_result_const.cpp
gcc62
comp_assign_result_const-gcc62.log
comp_bool1.cpp
gcc62
comp_bool1-gcc62.log
comp_bool2.cpp
gcc62
comp_bool2-gcc62.log
comp_bool_const1.cpp
gcc62
comp_bool_const1-gcc62.log
Common header used in some tests: common.h