[PATCH] Clean up client timeout definitions [rev. 2]

[Scott Lovenberg]

On Tue, Dec 4, 2012 at 8:26 PM, Jeremy Allison <jra at samba.org> wrote:
> On Tue, Dec 04, 2012 at 08:19:28PM -0500, simo wrote:
>> On Tue, 2012-12-04 at 16:42 -0800, Jeremy Allison wrote:
>> > On Tue, Dec 04, 2012 at 09:15:38AM -0500, scott.lovenberg at gmail.com wrote:
>> > > From: Scott Lovenberg <scott.lovenberg at gmail.com>
>> > >
>> > > The definitions for default client timeout values have been moved to client.h.  When initializing a client struct we use this value instead of the old hardcoded value.  The timeout value remains 20 seconds.
>> >
>> > I hate to say it but SHORT_TIMEOUT actually isn't
>> > used anywhere in the code that I can see, so it
>> > actually can be removed.
>>
>> git agrees:
>>
>> $ git grep SHORT_TIMEOUT
>> source3/client/client.c:#define SHORT_TIMEOUT (5*1000)
>
> Yeah, that;'s how I checked but git doesn't always
> tell the whole story - sometimes pidl autogenerated
> code can use things it doesn't know about :-).
>
> Jeremy.

Eek.  For future reference, is there any good way to check that I'm
not going to stomp on autogenereated code?

-- 
Peace and Blessings,
-Scott.