On Mon, Nov 10, 2014 at 13:51, Anthony J. Bentley wrote:
> Martin Natano writes:
>> Index: common/recover.c
>> ===================================================================
>> RCS file: /cvs/src/usr.bin/vi/common/recover.c,v
>> retrieving revision 1.15
>> diff -u -r1.15 recover.c
>> --- common/recover.c 27 Oct 2009 23:59:47 -0000      1.15
>> +++ common/recover.c 10 Nov 2014 17:10:05 -0000
>> @@ -339,15 +339,6 @@
>>      int fd;
>>      char *dp, *p, *t, buf[4096], mpath[MAXPATHLEN];
>>      char *t1, *t2, *t3;
>> -
>> -    /*
>> -     * XXX
>> -     * MAXHOSTNAMELEN is in various places on various systems, including
>> -     * <netdb.h> and <sys/socket.h>.  If not found, use a large default.
>> -     */
>> -#ifndef MAXHOSTNAMELEN
>> -#define     MAXHOSTNAMELEN  1024
>> -#endif
>>      char host[MAXHOSTNAMELEN];
>>
>>      gp = sp->gp;
> 
> FreeBSD nvi uses sysconf for this. If that's the right thing to do, we
> should probably synchronize with them...

overkill in my opinion. we're not going to change it.

diff applied. thanks.

Reply via email to