По моему - без разницы даже для команды разработчиков :)
Ну и по Occam's Razor - проще наверное юзать короткую запись if (ptr). Аргумент "возникает ошибочное ощущение что ptr - флаг" - это слабенький отмаз :) Если такое ощущение возникает - значит код еще незнаком и непонятен, а какая разница - непонятен смысл ветвелния по флагу или по указателю? :) Опять же флаг или указатель наверное будет видно по именованию?

ЗЫ: но все таки в перечне вариантов надо было if (!ptr) прописать вместо if (ptr) ;) Так сказать для равнозначности условий :)