Я делаю небольшой поиск утечки памяти в своей программе, но мой способ перегрузить новые и удалить (а также новые [] и удалить []) ничего не делает.
void* operator new (unsigned int size, const char* filename, int line)
{
void* ptr = new void[size];
memleakfinder.AddTrack(ptr,size,filename,line);
return ptr;
}
То, как я перегрузил new, показан в приведенном выше фрагменте кода. Я думаю, что-то с оператором, возвращающим void *, но я не знаю, что с этим делать.
void* ptr = new void[size];
Не могу этого сделать. Исправьте его.
Никогда не пытайтесь перегрузить новый/удалить глобально. Либо укажите их в базовом классе и выведите все объекты из этого класса, либо используйте пространство имен или параметр-распределитель шаблона. Почему, спросите вы. Потому что, если ваша программа больше одного файла и использует STL или другие библиотеки, вы собираетесь испортить.
Здесь дистиллированная версия оператора new из VS2005 new.cpp:
void * operator new(size_t size) _THROW1(_STD bad_alloc)
{ // try to allocate size bytes
void *p;
while ((p = malloc(size)) == 0)
if (_callnewh(size) == 0)
{ // report no memory
static const std::bad_alloc nomem;
_RAISE(nomem);
}
return (p);
}
RE:
Никогда не пытайтесь перегрузить новый/удалить глобально
Почему каждый раз, когда кто-то пытается использовать менее распространенную функцию С++, кто-то действует так, как это никогда не должно быть сделано?
Это делалось все время, это довольно часто, и я не работал в компании, которая этого не делала.
Глобально перегруженные новые и удаленные приложения чрезвычайно полезны для отслеживания памяти, ошибок памяти, переполнения буфера и т.д.
Никто в здравом уме не собирается проходить через программу с несколькими миллионами строк кода и добавляет новый и удаляемый элемент к каждому классу. Это просто глупо.
Возможно, вы можете сделать то, что хотите, с малой препроцессорной магией:
#include <iostream>
using namespace std;
void* operator new (size_t size, const char* filename, int line) {
void* ptr = new char[size];
cout << "size = " << size << " filename = " << filename << " line = " << line << endl;
return ptr;
}
#define new new(__FILE__, __LINE__)
int main() {
int* x = new int;
}
Я думаю, что проблема здесь в том, что ваш новый профиль параметров не соответствует стандартным новым операционным операторам, поэтому он не скрывается (и, тем самым, все еще используется).
Профили параметров для новых и удаления должны выглядеть так:
void* operator new(size_t);
void operator delete(void*, size_t);
Вы вызываете перегруженный оператор правильно, т.е. передаете ему дополнительные параметры?
Проблема связана с двумя аргументами, которые вы добавили к перегруженному новому оператору. Попробуйте сделать имя файла и строку глобальным в некотором роде (или переменные-члены, если вы перегружаете новые и удаляете для одного класса). Это должно работать лучше.