WW #12

[Дмитрий Свиридкин]

лучше завернуть всю имплементацию в namespace, чем указывать его в определениии каждого метода.

static int getTwoPow(int n) {
    double tmp = static_cast<double>(n);

Ну и зачем вам тут плавающая точка? Делите в целых числах. А лучше умножайте.

Оператор присваивания ломает size.

Код, который очищает вектор повторяется минимум четырежды. Создающий новый буфер -- тоже.
Его можно вынести в отдельную функцию. А лучше в структуру и переложить на ее дуструктор заботу об очистке в любой непонятной ситуации.

[Дмитрий Свиридкин]

include/my_vector_impl.h:57:9: error: cannot convert ‘my_vector::my_vector<int>’ to ‘my_vector::my_vector<int>::vector_impl&’
   57 |         holder.swap(other);
      |         ^~~~~~
include/my_vector_impl.h:27:55: note:   initializing argument 1 of ‘void my_vector::my_vector<T>::vector_impl::swap(my_vector::my_vector<T>::vector_impl&) [with T = int]’
   27 |     void my_vector<T>::vector_impl::swap(vector_impl& other) {

void my_vector<T>::push_back(const T &t) {
 if (holder.size_ == holder.capacity_ || holder.capacity_ == 1) {

Зачем здесь второе условие? Будет реаллокация пустого вектора с capacity == 1 -- а это неправильно.

push_back нарушает строгую гарантию исключений.

Оператор вывода выводит лишний пробел.

в reserve нужен move элементов

while((capacity - 1) & capacity) {
            capacity++;
        }

У вас пока самая медленная добивка до степени двойки. Попробуйте еще раз.

[Дмитрий Свиридкин]

pop_back можно сделать методом impl и переисполь в деструкторе, clear, resize

---

9.8