Классика жанра

[contradictor]

static inline void FREE(void* ptr) {
if (ptr != NULL) {
free(ptr);
ptr = NULL;
}
}

Comments

[feldgendler]

#define FREE free

[contradictor]

Гм.. что это лечит?

[feldgendler]

Ничего. Это просто 100% эквивалент тому, что написано в посте, хотя намерение автора было другое.

[contradictor]

В этом смысле - да.

[the_chiffa]

А вот если бы функция была определена как
static inline void FREE(void* const ptr )
все могло бы сложиться иначе!

[contradictor]

Тогда компилятор просто ругнулся бы на третью строчку, полагаю.

[yatur]

Мелкий пошел баг. Не эпичный. Да и не баг это вовсе. Компилятор же, небось, идиотское присваивание.
А что касается проверки на NULL - ну кто их знает - может, они миллионами NULL указатели освобождают в цикле. Сэкономил миллион вызовов free() - спас дерево. :)

[contradictor]

Компилятор же, небось, идиотское присваивание.
...удалит при оптимизации? Пользователю функции от этого не легче. Не говоря уже о том, что в случае повторного вызова с тем же prt наверняка прилетит SIGSEGV.

Лично я, как пользователь, был уверен - это дифайн (ну капсом же, да и логика подсказывает). Стоило это мне два часа отладки и разочарования в людях...

[yatur]

А, в этом смысле. Похоже, кому-то сказали, что дефайны зло, и их все надо заменить на инлайны. Ну он и заменил :)

Сделайте template и T*& ptr.

[contradictor]

Я заменил на дифайн - так проще и быстрее. В коде для эмбеддед систем не слишком приветствуют темплейты - растет размер.

ЗЫ и да, хидер - где была определена функция - используется еще и в pure C, так что передача по ссылке не годится.