[PATCH] adding custom options in radv protocol, strict ipv6 regex

[Ondrej Zajicek]

On Thu, Jul 27, 2023 at 03:38:27PM +0200, Alexander Zubkov wrote:
> Hi,
> 
> Have you had a chance to look at all this?

Hi

Sorry for keeping you wait, i finally got to this patchset and merged it.
I did some changes:

1) Removal of support for multiline string literals - the patch is simple
but it has an awkward consequence of making incomprehensible error messages
for unterminated strings. We discussed that and decided to drop it.

2) Refactor of bstrhextobin() and bstrbintohex() to be more generic and
not depend bstrtobyte16(), which was removed. I never liked it anyway.
Also, there is an explicit string of allowed delimiters in bstrhextobin(),
for from_hex(), instead of ignoring everything that is not a letter.

3) Change val_format() to not add 'hex:' prefix when printing bytestring.
It offers human readable, but not necessary back-parsable form (i.e more
like Scheme function 'display' than 'write'). That is consistent with its
behavior for strings, where quotation marks are also not added.

4) Use different approach for bytestring or string argument for password
keys, so the target expression can know which variant was used.

For other details, see the commits.

Thanks for the patchset. I especially like the cf_eval() / cf_eval_int()
changes.

-- 
Elen sila lumenn' omentielvo

Ondrej 'Santiago' Zajicek (email: santiago at crfreenet.org)
OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net)
"To err is human -- to blame it on a computer is even more so."