WW #12

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

src/main.cpp:863:27:   required from here
include/my_vector_impl.h:183:41: error: ‘malloc’ was not declared in this scope, and no declarations were found by argument-dependent lookup at the point of instantiation [-fpermissive]
  183 |    T* new_array = static_cast<T*>(malloc(n * sizeof(T)));
      |                                   ~~~~~~^~~~~~~~~~~~~~~

Подключать заголовки надо там, где вы используете их содержимое. У вас компилируется только потому, что в main my_Vector.h подключен после чего-то, подключившего malloc.h
В *_impl.h не надо подключать исходный заголовок.

чтобы не специфицировать namespace у каждого метода, лучше завернуть их определения все разом в namespace my_vector { .... }

	catch(...)
	{
		throw;
	}

И зачем?

void my_vector::my_vector<T>::push_back(const T &&t)
Не получится у вас тут помувать

push_back(T(std::move(nw[i]))); Зачем тут еще один вызов конструктора?

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

Вместо переноса push_backами, сделайте swap указателей. А лучше пусть это сделает оператор или конструктор перемещения.

В reserve надо делать move.

В конструкторе my_vector::my_vector<T>::my_vector(size_t n) деструктор вспомогательного вектора ничего не очистит.

Оператор присваивания нарушает строгую гарантию исключений.

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

5