Does this change really help anything? I'd rather had it consistent with the other call site, checking for non-zero return value and using -1 for DrawTextEx().