Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

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

ha4 lazy strings

Reported by: Sergei Shulman Owned by: Vladimir Rutsky
Priority: проверка Milestone:
Component: HA#4 lazy_string Version: 1.0
Keywords: Cc:

Description

Здравствуйте,

посмотрите, пожалуйста, реализацию ленивых строк.

С уважением,
Сергей

Change History (13)

comment:3 Changed 7 years ago by cpptester

Tested revision 708 by shulman.sergej.

Main tests

test

stage

result

info

log

run_clear_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_clear_no_mem-run.log

run_ctor_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_ctor_no_mem-run.log

run_empty.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_empty_eq.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_empty_eq-run.log

run_empty_wide.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_index_assign_copy_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_index_assign_copy_no_mem-run.log

run_swap_with_empty.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_swap_with_empty-run.log

Common header used in some tests: common.h

comment:4 Changed 7 years ago by cpptester

Tested revision 710 by shulman.sergej.

All tests passed. Good job!

comment:5 Changed 7 years ago by Sergei Shulman

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

comment:6 Changed 7 years ago by cpptester

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

Tested revision 710 by shulman.sergej.

Main tests

test

stage

result

info

log

run_assign_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_assign_c_str_no_mem-run.log

Common header used in some tests: common.h

comment:7 Changed 7 years ago by Vladimir Rutsky

Milestone: ha4-milestone1ha4-milestone2

Замечания:

  1. В каких случаях у вас data == nullptr? Мне кажется, что практически никогда, я рекомендую всегда хранить какой-то буфер там и избавиться от большого числа проверок data на nullptr.

Ещё скоро придёт отчет о некоторых новых падающих тестах.

comment:8 Changed 7 years ago by Vladimir Rutsky

  1. Используйте <cstddef> вместо <stddef.h>.

comment:9 Changed 7 years ago by cpptester

Tested revision 710 by shulman.sergej.

Main tests

test

stage

result

info

log

run_assign_c_str_no_mem.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_assign_c_str_no_mem-run.log

run_char_traits_custom_eq.cpp

gcc62

FAILURE

run_char_traits_custom_eq-gcc62.log

run_index_const_wstr.cpp

gcc62

SUCCESS

run

FAILURE

exit code 139

run_index_const_wstr-run.log

Common header used in some tests: common.h

comment:10 Changed 7 years ago by cpptester

Tested revision 912 by shulman.sergej.

All tests passed. Good job!

comment:11 Changed 7 years ago by Sergei Shulman

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

Замечания учёл. Код для прохождения новых тестов доработал, попутно немного сократив.

comment:12 Changed 7 years ago by Vladimir Rutsky

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

Замечания:

  1. Вместо собственной реализации подсчета ссылок я бы рекомендовал использовать std::shared_ptr<storage>.

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

comment:13 Changed 7 years ago by Vladimir Rutsky

Milestone: ha4-milestone2

Milestone ha4-milestone2 deleted

Note: See TracTickets for help on using tickets.