Opened 6 years ago

Closed 6 years ago

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

HW4

Reported by: Igor Ryabtsev Owned by: rutsky,grabovoy.philipp
Priority: проверка Milestone: ha4-deadline
Component: HA#4 function Version: 1.0
Keywords: Cc:

Description

Прошу проверить 4 домашнее задание.

Change History (10)

comment:1 Changed 6 years ago by cpptester

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

Tested revision 595 by ryabtsev.igor.

Main tests

test

stage

result

info

log

run_assign.cpp

SUCCESS
FAILURE

exit code 139

run_assign-run.log

run_free_function.cpp

SUCCESS
FAILURE

exit code 139

run_free_function-run.log

Common header used in some tests: common.h

comment:2 Changed 6 years ago by Igor Ryabtsev

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

comment:3 Changed 6 years ago by cpptester

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

Tested revision 651 by ryabtsev.igor.

Smoke tests

test

stage

result

info

log

run_function_smoke_test.cpp

WARNING

run_function_smoke_test-gcc62.log

SUCCESS
SUCCESS

run_function_smoke_test-valgrind.log

WARNING

run_function_smoke_test-gcc62-asan.log

SUCCESS
WARNING

run_function_smoke_test-clang-san.log

SUCCESS

Common header used in some tests: common.h

comment:4 Changed 6 years ago by Igor Ryabtsev

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

comment:5 Changed 6 years ago by cpptester

Tested revision 652 by ryabtsev.igor.

All tests passed. Good job!

comment:6 Changed 6 years ago by Vladimir Rutsky

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

Замечания:

  1. Используйте std::unique_ptr для хранения обёрнутого функтора, вместо std::shared_ptr --- вам достаточно единоличного владения обернутом функтором.
  1. В store_basis необходимо определить виртуальный деструктор. В общем случае его отсутствие приводит к тому, что деструктор storage не будет вызван, но в вашем случае этого не проиходит, т.к. std::shared_ptr сконструированный от производного класса запоминает делитер для оригинального типа.

comment:7 Changed 6 years ago by Igor Ryabtsev

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

comment:8 Changed 6 years ago by cpptester

Tested revision 739 by ryabtsev.igor.

All tests passed. Good job!

comment:9 Changed 6 years ago by Igor Ryabtsev

Изменил на std::unique_ptr и добавил виртуальный деструктор.

comment:10 Changed 6 years ago by Vladimir Rutsky

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

Решение зачтено.

Note: See TracTickets for help on using tickets.