A NULL defname would early exit (doing nothing) this isn't good behavior as this function should always make the name unique and a NULL defname is likely an error in the code which would allow duplicate names. This is also inconsistent with BLI_uniquename_cb which always wrote the defname into the name if it was empty. Mark this argument as never-NULL.