Opened 6 years ago
Closed 6 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 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:2 Changed 6 years ago by
Я создал директорию для вас в cpp17 и скопировал туда ваше решение из cpp16, пожалуйста склонируйте репозиторий cpp17 и исправьте решение там:
svn checkout http://trac.compscicenter.ru/svn/cpp17/golovanov.egor/ --username golovanov.egor
comment:3 Changed 6 years ago by
Tested revision 645 by golovanov.egor.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
FAILURE | ||||
SUCCESS | ||||
FAILURE | exit code 139 | |||
SUCCESS | ||||
FAILURE | exit code 139 | |||
Common header used in some tests: common.h
comment:4 Changed 6 years ago by
Tested revision 671 by golovanov.egor.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
FAILURE | ||||
SUCCESS | ||||
FAILURE | exit code 139 | |||
Common header used in some tests: common.h
comment:6 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:7 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Tested revision 672 by golovanov.egor.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
SUCCESS | ||||
FAILURE | exit code 139 | |||
Common header used in some tests: common.h
comment:8 Changed 6 years ago by
Tested revision 672 by golovanov.egor.
Main tests
test | stage | result | info | log |
---|---|---|---|---|
SUCCESS | ||||
FAILURE | exit code 139 | |||
Common header used in some tests: common.h
comment:10 Changed 6 years ago by
Tested revision 734 by golovanov.egor.
All tests passed. Good job!
comment:11 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:12 Changed 6 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
Привет!
Несколько косметических замечаний:
- В
function(function &&f)
нет необходимости мануально обнулятьinvoker_p (unique_ptr)
после мува: по стандарту он уже будет таким (https://stackoverflow.com/a/36071266) func_invoker::clone
может сразу возвращатьunique_ptr<invoker>
(вместоinvoker*
) — тогда при написании нового кода можно будет создать меньше ситуаций, ведущих к учетке памяти. Пример тот самый, с двумяnew
для аргументов ф-ии:foo(invoker_p->clone(), f.invoker_p->clone())
comment:13 Changed 6 years ago by
Tested revision 825 by golovanov.egor.
Smoke tests
test | stage | result | info | log |
---|---|---|---|---|
FAILURE | ||||
Common header used in some tests: common.h
comment:14 Changed 6 years ago by
Tested revision 826 by golovanov.egor.
All tests passed. Good job!
comment:15 Changed 6 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|
comment:17 Changed 6 years ago by
Tested revision 827 by golovanov.egor.
All tests passed. Good job!
comment:19 Changed 6 years ago by
Resolution: | → задача сдана |
---|---|
Status: | new → closed |
Tested revision 631 by golovanov.egor.
Smoke tests
test
stage
result
info
log
run_function_smoke_test.cpp
exit code 139
run_function_smoke_test-run.log
Common header used in some tests: common.h