CPP-KT / bimap-task

0 stars 0 forks source link

Added use after move tests #16

Closed Ismaxis closed 1 month ago

Ismaxis commented 1 year ago

Added usage of moved comparator checks

Ismaxis commented 1 year ago

хмм, а как вообще может получиться, что у тебя тут будет использоваться помуванный компаратор?

Пока я писал у меня случился баг, что я в мув конструкторе бимапа сначала мувал компараторы, а потом свопал деревья (где тоже случался обратный мув компараторов). Убрать изначальный мув компараторов невозможно, т.к они могут быть не default constructible, поэтому мне для решения потребовалось ещё раз хорошенько подумать, что означает мув бимапы.

Мне показалось, что стоит добавить такой тест, т.к. можно легко не подумав сделать баг, подобный моему, и мапа не будет работать со сложными компараторами

Dogzik commented 11 months ago

Проверь, нельзя ли связать твой тест с существующим issue и расширить

Ismaxis commented 11 months ago

Проверь, нельзя ли связать твой тест с существующим issue и расширить

16

Added: 84e4d67

dedlocc commented 1 month ago

Спасибо, взято за основу для новых тестов