Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#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 Andrey Chausov

Cc: dxahtepb@… added
Version: 1.0

comment:2 Changed 6 years ago by cpptester

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

comment:3 Changed 6 years ago by Andrey Chausov

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

comment:4 Changed 6 years ago by cpptester

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

comment:5 Changed 6 years ago by Andrey Chausov

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

comment:6 Changed 6 years ago by cpptester

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

Tested revision 335 by chausov.andrej.

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 Andrey Chausov

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

comment:8 Changed 6 years ago by cpptester

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

Tested revision 337 by chausov.andrej.

Main tests

test

stage

result

info

log

comp_unique_const.cpp

gcc62

FAILURE

comp_unique_const-gcc62.log

ncomp_incomplete_type1.cpp

gcc62

FAILURE

ncomp_incomplete_type2.cpp

gcc62

FAILURE

Common header used in some tests: common.h

comment:9 Changed 6 years ago by Andrey Chausov

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

comment:10 Changed 6 years ago by cpptester

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

Tested revision 341 by chausov.andrej.

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:11 Changed 6 years ago by Andrey Chausov

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

comment:12 Changed 6 years ago by cpptester

Tested revision 343 by chausov.andrej.

All tests passed. Good job!

comment:13 Changed 6 years ago by Vladimir Rutsky

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

Замечания:

  1. bool is_convertable(U * ptr) не во всех путях исполнения возвращает значение.
  1. assert(is_convertable(data)); проверяется во время выполнения, хотя это можно сделать во время компиляции. Плюс, с включенным макросом NDEBUG, assert будет скомпилирован как void, и проверка не выполнится.
  1. Проверка is_convertable излишня: после проверки вы делаете присвоение this->data = data;, которое скомпилируется только тогда, когда типы неявно приводимы.

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

comment:14 Changed 6 years ago by cpptester

Tested revision 498 by chausov.andrej.

Smoke tests

test

stage

result

info

log

run_linked_ptr_gen_smoke_test.cpp

FAILURE

run_linked_ptr_gen_smoke_test-gcc62.log

run_smoke_test.cpp

FAILURE

run_smoke_test-gcc62.log

Common header used in some tests: common.h

comment:15 Changed 6 years ago by Andrey Chausov

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

comment:16 Changed 6 years ago by cpptester

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

Tested revision 499 by chausov.andrej.

Smoke tests

test

stage

result

info

log

run_linked_ptr_gen_smoke_test.cpp

FAILURE

run_linked_ptr_gen_smoke_test-gcc62.log

run_smoke_test.cpp

FAILURE

run_smoke_test-gcc62.log

Common header used in some tests: common.h

comment:18 Changed 6 years ago by cpptester

Tested revision 514 by chausov.andrej.

All tests passed. Good job!

comment:19 Changed 6 years ago by Vladimir Rutsky

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

is_convertable() теперь не используется и её стоит удалить.

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

comment:20 Changed 6 years ago by cpptester

Tested revision 518 by chausov.andrej.

All tests passed. Good job!

Note: See TracTickets for help on using tickets.