Peter Wemm writes:
>Gary Jennejohn wrote:
>> Bruce Evans writes:
>> >On Mon, 3 Apr 2000, Gary Jennejohn wrote:
>> >
>> >> Bruce Evans writes:
>> >> >On Sun, 2 Apr 2000, Gary Jennejohn wrote:
>> >> >> I think we should nuke csu_hdr since it's not used anywhere. Is that
>> >> >> what you really mean ?
>> >> >
>> >> >Yes. I'm trying the following patch. Only tested at compile time.
>> >> >
>> >> [patch snipped]
>> >>
>> >> Thank you, Bruce ! This is pretty much the same patch I tested.
>> >>
>> >> So, should I commit it ?
>> >
>> >If you have tested it :-).
>> >
>>
>> I'm running with the change right now. No problems.
>
>I would prefer that we did this:
>
>#define MAX_LEN (min(128, MLEN))
>
>or something like that. This should stop Bad Things happening if
>somebody recompiles with MLEN set specifically to 128 (and is an ideal
>MFC candidate for 4.x for when people set MLEN to 256 over there).
>
This is a pretty good idea, too. But I already deleted csu_hdr in -current.
It would be easy enough to back out the change if there's consensus.
---
Gary Jennejohn / [EMAIL PROTECTED] [EMAIL PROTECTED]
To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message
