#823 closed ожидается проверка (задача сдана)
WW #12 smirnov.ilya
| Reported by: | smirnov.ilya | Owned by: | Дмитрий Лапшин (lapshin) |
|---|---|---|---|
| Component: | WW_vector | Version: | |
| Keywords: | Cc: |
Description
Change History (11)
comment:1 Changed 4 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to smirnov.ilya |
|---|---|
| Summary: | WW #12 → WW #12 smirnov.ilya |
| Type: | ожидается проверка → ожидаются исправления |
| Version: | 1.0 |
comment:3 Changed 4 years ago by
| Owner: | changed from smirnov.ilya to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
comment:4 Changed 4 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to smirnov.ilya |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
А ещё всё-таки ../include запрещен.
Работать особо не хочет, валгринд орёт, крашится. Читать код с трейтами и аллокаторами так весело, что я пожалуй пас разбираться с чего бы, но очень пахнет, что ты забыл создавать объекты где-то.
Название garbage_collector оч странное. И по её использованию ни хрена это не сильная гарантия.
Чини.
comment:5 Changed 4 years ago by
А можно пожалуйста посмотреть, на что валгринд ругается? У меня почему-то он молчит :(
comment:6 Changed 4 years ago by
| Owner: | changed from smirnov.ilya to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
Кажется я уже просто ничего не выделяю... Надеюсь, сейчас валгринд не закричит
comment:8 Changed 4 years ago by
| Owner: | changed from Дмитрий Лапшин (lapshin) to smirnov.ilya |
|---|---|
| Type: | ожидается проверка → ожидаются исправления |
Тесты починились, и стали жёстче.
clean не комплиируется на классах без конструктора по умолчанию.
comment:9 Changed 4 years ago by
| Owner: | changed from smirnov.ilya to Дмитрий Лапшин (lapshin) |
|---|---|
| Type: | ожидаются исправления → ожидается проверка |
comment:10 Changed 4 years ago by
| Resolution: | → задача сдана |
|---|---|
| Status: | assigned → closed |
Так. За играми с аллокатором:
- Копирующий конструктор, который делает дефолт-инициализацию (а вдруг нет конструктора по умолчанию?) а потом копирующе присваивает не канает.
- Из-за этого и
operator=ведёт себя плохо. - reserve копирует вместо перемещения, поэтому у него нет строгой гарантии.
- И у resize из-за этого тоже нет. Но если починить reserve не хватит, некидающий default ctor не обещали.
Короче, переусложнил и не работает при этом.
6.
comment:11 Changed 4 years ago by
То есть в 4 пункте надо было добавлять default ctor-ом объекты в копию, а потом свопаться ?
А в пункте 1 я не понял : у нас ведь по условию есть конструктор по умолчанию ... Или я не прав ?
Структура файлов для кого?
Работает очень плохо, крашится, валгринд, всё вместе.