[PATCH] Remove x_file from ntlm_auth and ntlm_auth4 (was: Re: [PATCH] some cleanups)

[Jeremy Allison]

On Fri, Nov 25, 2016 at 02:22:06PM +1300, Andrew Bartlett wrote:
> On Thu, 2016-11-24 at 20:45 +0100, Volker Lendecke wrote:
> > Hi!
> > 
> > Review appreciated!
> > 
> 
> G'Day Volker,
> 
> Could you please use a relevant subject, like [PATCH] Remove x_file
> from ntlm_auth and ntlm_auth4?
> 
> It is really hard to follow which clean-ups are which, and it is a
> large code-base.  Most of your work is great, but it is frustrating to
> need to open the patch to work out what it is on.
> 
> As to x_file, xfile.c mentions:
> 
>   stdio is very convenient, but on some systems the file descriptor
>   in FILE* is 8 bits, so it fails when more than 255 files are open. 
> 
> The history (from memory) was that the 'some systems' referenced some
> ancient solaris.  Thankfully it looks like this is long-gone, for
> Solaris 10 and later:
> 
> http://www.oracle.com/technetwork/server-storage/solaris/stdio-256-1366
> 98.html

ntlm_auth4 is gone now, so 1 and 2 are no longer relevent.

Pushed patch 3 with updated subject line and comment explaining
cleanup.

Cheers,

	Jeremy.