Opened 4 years ago

Closed 4 years ago

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

HW 4 Aleksandr Eslikov

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

Description


Change History (9)

comment:1 Changed 4 years ago by cpptester

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

Tested revision 603 by eslikov.aleksandr.

Smoke tests

test

stage

result

info

log

run_function_smoke_test.cpp

SUCCESS
SUCCESS
SUCCESS

run_function_smoke_test-valgrind.log

SUCCESS
SUCCESS
WARNING

run_function_smoke_test-clang-san.log

SUCCESS

Common header used in some tests: common.h

comment:2 Changed 4 years ago by cpptester

Tested revision 637 by eslikov.aleksandr.

Main tests

test

stage

result

info

log

run_free_function.cpp

SUCCESS
FAILURE

exit code 139

run_free_function-run.log

Common header used in some tests: common.h

comment:3 Changed 4 years ago by cpptester

Tested revision 648 by eslikov.aleksandr.

All tests passed. Good job!

comment:4 Changed 4 years ago by cpptester

Tested revision 649 by eslikov.aleksandr.

All tests passed. Good job!

comment:5 Changed 4 years ago by Aleksandr Eslikov

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

comment:6 Changed 4 years ago by Филипп

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

Привет!

Несколько замечаний:

  1. Наследоваться от std::exception нужно публично
  2. Для FunctionWrapperInterface shared_ptr избыточен, достаточно unique_ptr

comment:7 Changed 4 years ago by cpptester

Tested revision 812 by eslikov.aleksandr.

All tests passed. Good job!

comment:8 Changed 4 years ago by Aleksandr Eslikov

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

Привет!

Поправил.

comment:9 Changed 4 years ago by Филипп

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

И в самом FunctionWrapperInterface::clone явно приводить результат к типу smart-поинтера необязательно: компилятор применит неявное преобразование, исходя из типа возвращаемого функцией значения (пока оно не auto).

Note: See TracTickets for help on using tickets.