#157 closed ожидаются исправления (задача сдана)
HA-5
| Reported by: | Волков Даниил | Owned by: | Vladimir Rutsky |
|---|---|---|---|
| Priority: | проверка | Milestone: | |
| Component: | HA#5 function | Version: | 1.0 |
| Keywords: | std::function, function | Cc: |
Description
Implements std::function interface
Please, check my work.
Change History (9)
comment:1 Changed 7 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
comment:2 Changed 7 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|
comment:3 Changed 7 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
Tested revision 1011 by volkov.daniil.
Smoke tests
test | stage | result | info | log |
|---|---|---|---|---|
gcc62 | WARNING | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | WARNING | |||
asan | SUCCESS | |||
clang-san | FAILURE | |||
Common header used in some tests: common.h
comment:4 Changed 7 years ago by
| Type: | ожидаются исправления → ожидается проверка |
|---|
comment:5 Changed 7 years ago by
| Type: | ожидается проверка → ожидаются исправления |
|---|
Tested revision 1014 by volkov.daniil.
Smoke tests
test | stage | result | info | log |
|---|---|---|---|---|
gcc62 | WARNING | |||
run | SUCCESS | |||
valgrind | SUCCESS | |||
gcc62-asan | WARNING | |||
asan | SUCCESS | |||
clang-san | SUCCESS | |||
san | SUCCESS | |||
Common header used in some tests: common.h
comment:6 Changed 7 years ago by
Замечания:
- Не используйте
operator ,для разделения команд, когда можно использовать блок команд, разделённых точкой с запятой:
if (sizeof(functor_holder<F>) < BUFFER_SIZE)
fun_type = SMALL_FUNCTOR, new(small_functor) functor_holder<F>(fun);
else
pfholder_.reset(new functor_holder<F>(fun)), fun_type = FUNCTION;
обычно на одной строке одна команда, и ваш код хуже читается, чем:
if (sizeof(functor_holder<F>) < BUFFER_SIZE)
{
fun_type = SMALL_FUNCTOR;
new(small_functor) functor_holder<F>(fun);
}
else
{
pfholder_.reset(new functor_holder<F>(fun));
fun_type = FUNCTION;
}
comment:8 Changed 7 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | new → closed |
- Нет необходимости делать
defineна какое-то конкретное значение:
#define FN_HPP 1
достаточно, просто:
#define FN_HPP
Решение зачтено.
Note: See
TracTickets for help on using
tickets.
Tested revision 933 by volkov.daniil.
Smoke tests
test
stage
result
info
log
run_function_smoke_test.cpp
gcc62
run_function_smoke_test-gcc62.log
run
valgrind
run_function_smoke_test-valgrind.log
gcc62-asan
run_function_smoke_test-gcc62-asan.log
asan
clang-san
run_function_smoke_test-clang-san.log
Common header used in some tests: common.h