#300 closed ожидается проверка (задача сдана)
WW #4
Reported by: | Vavilov Mark | Owned by: | Sokolov Viacheslav |
---|---|---|---|
Component: | WW_mergesort | Version: | 2.0 |
Keywords: | Cc: |
Description
Change History (5)
comment:1 Changed 5 years ago by
Summary: | WW #3 → WW #4 |
---|
comment:2 Changed 5 years ago by
Type: | ожидается проверка → ожидаются исправления |
---|
comment:3 Changed 5 years ago by
Type: | ожидаются исправления → ожидается проверка |
---|---|
Version: | 1.0 → 2.0 |
comment:4 Changed 5 years ago by
Resolution: | → задача сдана |
---|---|
Status: | assigned → closed |
В компараторах потеряна константность (происходит каст к неконстантным указателям).
Строку "Memory allocation failed" стоило бы именовать и вынести отдельно, чтобы не дублировать (вдруг захочется поменять потом?)
В mergesort.h отсутствует проверка контрактов (ненулевые указатели, ненулевой element_size, непонятно, можно ли передавать 0 элементов и тд)
В реализации merge много дублирований кода, их можно было бы избежать
Удачное решение - выделить буфер заранее на все время работы.
comment:5 Changed 5 years ago by
Бонусное задание выполнено, но в случае, если внезапно память кончилось при аллокации в main, пользователь не увидит об этом сообщения
Note: See
TracTickets for help on using
tickets.
Не компилируется