Even more warning fixes

[Jamey Sharp]

On Fri, Nov 12, 2010 at 10:21 AM, Cyril Brulebois <kibi at debian.org> wrote:
> Jamey Sharp <jamey at minilop.net> (12/11/2010):
>> Oh, wait. You don't need the conditionals or the lookup_ret variable
>> here at all: the result pointer is already guaranteed to be NULL if
>> the return value is not Success.
>
> I guess dix/deprecated.c should be adjusted accordingly, then?

I guess so, yeah.

...Also it looks like the code that's supposed to emit a warning if
you call those deprecated functions can't ever do so? --warn==0 so the
if condition is always false...

Jamey