Имхо, это ничем не плохо и в некоторых редких ситуациях удобно. Ровно как и оригинальный вариант - ничем не плох и в некоторых ситуациях удобен. Лично по мне, всё это "улучшение" сугубо теоретическое, на практике FreeAndNil более чем хорош, проблем с ним я в жизни не видел, и чтобы их организовать, нужно такое выдающееся головотяпство, что, мягко говоря, не на зеркало стоит пенять.

Месяца через три после того, как я в первый раз стал начальником отдела, я встал перед необходимостью уволить двоих программистов. На их примере я выработал простой принцип: не брать себе в подчинение тех людей, которых не я собеседовал и не я выбирал. С тех пор, то есть уже где-то шестнадцать лет, я не уволил ни одного человека. Необходимости не было. Видимо, тех, с кем она возникла бы, я отсеиваю ещё на собеседовании. Очень везучий.


Это да, такими странными вещами я не занимаюсь. В том числе поэтому мне и везёт.


Если налить в чайник раствор серной кислоты и прокипятить, довольно велик риск неприятных последствий. Но это не значит, что чайник плох и приводит к багам.

Да улучшила и улучшила, почему бы и нет. Ну, может, отвалится какой-нибудь код двадцатилетней давности, типа FreeAndNil(TreeNode.Data), так, мягко говоря, не жалко. Просто, имхо, здесь нечего обсуждать на три страницы.

Думаю, прокатит. И это будет хорошо и правильно.


Я лет двадцать не смотрел в декларацию FreeAndNil. Мои воспоминания о том, что там просто бестиповая переменная, отстали от жизни? Насколько я помню, var мешает употребить там TObject.

Нечего думать. Необнулённый указатель гораздо хуже.