Opened 4 years ago

Closed 4 years ago

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

ha4

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

Description

Здравствуйте!
Задание 4.
При сдаче возникла проблема - нет прав к репозиторию 2017 года, т.к. заканчивал 1 часть в 2016. Поэтому задание залил туда и присоединил архив на сайте CSC. Извините, стоило раньше отправить задание, чтобы заметить эту ошибку.

Change History (19)

comment:1 Changed 4 years ago by cpptester

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

Tested revision 631 by golovanov.egor.

Smoke tests

test

stage

result

info

log

run_function_smoke_test.cpp

SUCCESS
FAILURE

exit code 139

run_function_smoke_test-run.log

Common header used in some tests: common.h

comment:2 Changed 4 years ago by Vladimir Rutsky

Я создал директорию для вас в cpp17 и скопировал туда ваше решение из cpp16, пожалуйста склонируйте репозиторий cpp17 и исправьте решение там:

svn checkout http://trac.compscicenter.ru/svn/cpp17/golovanov.egor/ --username golovanov.egor

comment:3 Changed 4 years ago by cpptester

Tested revision 645 by golovanov.egor.

Main tests

test

stage

result

info

log

comp_rvalue_argument.cpp

FAILURE

comp_rvalue_argument-gcc62.log

run_free_function.cpp

SUCCESS
FAILURE

exit code 139

run_free_function-run.log

run_functor_copied.cpp

SUCCESS
FAILURE

exit code 139

run_functor_copied-run.log

Common header used in some tests: common.h

comment:4 Changed 4 years ago by cpptester

Tested revision 671 by golovanov.egor.

Main tests

test

stage

result

info

log

comp_rvalue_argument.cpp

FAILURE

comp_rvalue_argument-gcc62.log

run_free_function.cpp

SUCCESS
FAILURE

exit code 139

run_free_function-run.log

Common header used in some tests: common.h

comment:5 Changed 4 years ago by cpptester

Tested revision 672 by golovanov.egor.

All tests passed. Good job!

comment:6 Changed 4 years ago by golovanov.egor

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

comment:7 Changed 4 years ago by cpptester

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

Tested revision 672 by golovanov.egor.

Main tests

test

stage

result

info

log

run_functor_desctructor.cpp

SUCCESS
FAILURE

exit code 139

run_functor_desctructor-run.log

Common header used in some tests: common.h

comment:8 Changed 4 years ago by cpptester

Tested revision 672 by golovanov.egor.

Main tests

test

stage

result

info

log

run_functor_desctructor.cpp

SUCCESS
FAILURE

exit code 139

run_functor_desctructor-run.log

Common header used in some tests: common.h

comment:9 Changed 4 years ago by cpptester

Tested revision 672 by golovanov.egor.

All tests passed. Good job!

comment:10 Changed 4 years ago by cpptester

Tested revision 734 by golovanov.egor.

All tests passed. Good job!

comment:11 Changed 4 years ago by golovanov.egor

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

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

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

Привет!

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

  1. В function(function &&f) нет необходимости мануально обнулять invoker_p (unique_ptr) после мува: по стандарту он уже будет таким (https://stackoverflow.com/a/36071266)
  2. func_invoker::clone может сразу возвращать unique_ptr<invoker> (вместо invoker*) — тогда при написании нового кода можно будет создать меньше ситуаций, ведущих к учетке памяти. Пример тот самый, с двумя new для аргументов ф-ии: foo(invoker_p->clone(), f.invoker_p->clone())

comment:13 Changed 4 years ago by cpptester

Tested revision 825 by golovanov.egor.

Smoke tests

test

stage

result

info

log

run_function_smoke_test.cpp

FAILURE

run_function_smoke_test-gcc62.log

Common header used in some tests: common.h

comment:14 Changed 4 years ago by cpptester

Tested revision 826 by golovanov.egor.

All tests passed. Good job!

comment:15 Changed 4 years ago by golovanov.egor

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

comment:16 Changed 4 years ago by golovanov.egor

Привет!

Замечания исправил.

comment:17 Changed 4 years ago by cpptester

Tested revision 827 by golovanov.egor.

All tests passed. Good job!

comment:18 Changed 4 years ago by golovanov.egor

Теперь точно все)

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

Resolution: задача сдана
Status: newclosed
Note: See TracTickets for help on using tickets.