#98 closed ожидается проверка (задача сдана)
ДЗ#3: linked_pointer
Reported by: | Viacheslav Kukushkin | Owned by: | Vladimir Rutsky |
---|---|---|---|
Priority: | проверка | Milestone: | |
Component: | HA#3 linked_ptr | Version: | |
Keywords: | Cc: |
Description
Проверьте, пожалуйста
Change History (25)
comment:1 Changed 7 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 7 years ago by
Tested revision 541 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
Common header used in some tests: common.h
comment:3 Changed 7 years ago by
Tested revision 541 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
Common header used in some tests: common.h
comment:4 Changed 7 years ago by
Вячеслав, теперь в логе есть стек трейс, где видно как именно происходит разыменование нулевого указателя.
comment:5 Changed 7 years ago by
Tested revision 549 by kukushkin.vyacheslav.
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 7 years ago by
Tested revision 552 by kukushkin.vyacheslav.
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:7 Changed 7 years ago by
Tested revision 567 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
Common header used in some tests: common.h
comment:8 Changed 7 years ago by
Tested revision 568 by kukushkin.vyacheslav.
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:9 Changed 7 years ago by
Tested revision 569 by kukushkin.vyacheslav.
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:10 Changed 7 years ago by
Tested revision 600 by kukushkin.vyacheslav.
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:11 Changed 7 years ago by
Tested revision 601 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
gcc62 | SUCCESS | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | SUCCESS | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | WARNING | |||
Common header used in some tests: common.h
comment:12 Changed 7 years ago by
Tested revision 602 by kukushkin.vyacheslav.
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:13 Changed 7 years ago by
Tested revision 604 by kukushkin.vyacheslav.
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:14 Changed 7 years ago by
Tested revision 605 by kukushkin.vyacheslav.
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:15 Changed 7 years ago by
Tested revision 606 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:16 Changed 7 years ago by
Tested revision 607 by kukushkin.vyacheslav.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:17 Changed 7 years ago by
Tested revision 617 by kukushkin.vyacheslav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:18 Changed 7 years ago by
Tested revision 619 by kukushkin.vyacheslav.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
gcc62 | FAILURE | |||
gcc62 | FAILURE | |||
Common header used in some tests: common.h
comment:19 Changed 7 years ago by
Tested revision 632 by kukushkin.vyacheslav.
All tests passed. Good job!
comment:20 Changed 7 years ago by
Tested revision 634 by kukushkin.vyacheslav.
All tests passed. Good job!
comment:21 Changed 7 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:22 Changed 7 years ago by
Resolution: | → задача сдана |
---|---|
Status: | new → closed |
я так и не понял, почему get() должен быть const (а ну как пользователь удалит data ?)
но иначе тесты не проходят
T* get() const {
return data;
}
Константность get()
необходима, чтобы пользователь мог получить значение хранимого указателя, даже если объект linked_ptr
константный.
Решение зачтено.
comment:23 Changed 7 years ago by
Бонусное задание не выполнено.
Тест для бонусного задания (должен компилироваться без предупреждений):
#include "linked_ptr.hpp" struct C; int main() { // Should compile. Completeness of C is not needed. smart_ptr::linked_ptr<C> p1; smart_ptr::linked_ptr<C> p2(p1); // Shouldn't compile: //smart_ptr::linked_ptr<C> p3(reinterpret_cast<C *>(123)); //p1.reset(reinterpret_cast<C *>(123)); }
comment:25 Changed 6 years ago by
Tested revision 634 by kukushkin.vyacheslav.
All tests passed. Good job!
Smoke tests
test
stage
result
info
log
run_linked_ptr_gen_smoke_test.cpp
gcc62
run_linked_ptr_gen_smoke_test-gcc62.log
run
valgrind
run_linked_ptr_gen_smoke_test-valgrind.log
gcc62-asan
run_linked_ptr_gen_smoke_test-gcc62-asan.log
asan
clang-san
run_linked_ptr_gen_smoke_test-clang-san.log
san
run_linked_ptr_gen_smoke_test-san.log
run_smoke_test.cpp
gcc62
run_smoke_test-gcc62.log
run
valgrind
run_smoke_test-valgrind.log
gcc62-asan
run_smoke_test-gcc62-asan.log
asan
clang-san
run_smoke_test-clang-san.log
san
run_smoke_test-san.log
Common header used in some tests: common.h