Замечание по классу CMyStack

[alexey-malov]

    ~CMyStack()
    {
        Clear();
        free(m_top);
    }

• [x] free можно вызывать только для освобождения памяти, полученной через malloc, calloc или realloc. Если указатель был получен любым иным способом, использовать функцию free с ним нельзя

[alexey-malov]

    CMyStack<T>& operator=(CMyStack&& stack)
    {
        if (this != &stack)
        {
            this->m_pTop = stack.m_pTop;
        }
        return *this;
    }

• [ ] Данные, на которые указывал стек-приёмник освобождены не будут и произойдёт утечка памяти
• [ ] И стек-приёмник, и стек-источник будут указывать на одну и ту же вершину. В результате при их разрушении будет неопределённое поведение, когда стек, деструктор которого вызвался вторым, начнёт "удалять" уже удалённые деструктором первого стека данные

[alexey-malov]

    CMyStack<T>& operator=(CMyStack&& stack)
    {
        if (this != &stack)
        {
            Clear();
            m_top = stack.m_top;
            stack = nullptr;
        }
        return *this;
    }

• [ ] Нет теста, код не компилируется, иначе код бы не скомпилировался - нельзя присвоить nullptr стеку