genproto has difficulty parsing some NIFs

[lewisr]

While installing ArcaOS on Hawking, I was forced to use the Compaq-supplied B57 NIC driver (neither the MMBGE nor the Broadcom reference driver work correctly on that hardware). During boot, GENPROTO complained about the duplex setting in the NIF:

Nif-parser: B57.nif#58: String "HALF" is longer that allowed StrLength.

This comes from GENPROTO.PAS.

For its part of the incident, the NIF actually says:

[Duplex]
tag = DUPLEX
display = "Duplex Mode"
type = string
strlength = 3
set = HALF,FULL
optional = YES
editable = YES
help = "This parameter specifies duplex mode on the ethernet controller.  This parameter is optional unless the Linespeed parameter is specified.  The Linespeed parameter must be set when this keyword is used.  If neither the Duplex nor the Linespeed paramaters are specified the ethernet controller will default to autonegotiate mode."

(including the misspelling of "parameters" in the last sentence)

[lewisr]

Actually, this is WAD.

The NIF is indeed in error:

strlength = 3
set = HALF,FULL

Changing strlength to 4 addresses the problem. IMO, no change should be needed to genproto.