[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#54352] [PATCH] services: dnsmasq: Add more options.
From: |
Maxime Devos |
Subject: |
[bug#54352] [PATCH] services: dnsmasq: Add more options. |
Date: |
Mon, 21 Mar 2022 19:36:41 +0100 |
User-agent: |
Evolution 3.38.3-1 |
Ludovic Courtès schreef op ma 21-03-2022 om 16:22 [+0100]:
> I think this suggestion is beyond the scope of this review: we’ve never
> used sanitizers like this before (or almost), and this particular piece
> of code doesn’t use them.
>
> Also, with the recent discussion about the introduction of contracts,
> I’d rather wait an use contracts everywhere once they’re available.
Seems reasonable to me, given that the specifics weren't discussed yet,
although _everywhere_ (for all procedures, records, ...) seems a bit
much, unless you meant every field of the dnsmasq record.
Greetings,
Maxime.
signature.asc
Description: This is a digitally signed message part
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/12
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Ludovic Courtès, 2022/03/19
- [bug#54352] [PATCH v2] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/20
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Ludovic Courtès, 2022/03/21
- [bug#54352] [PATCH] services: dnsmasq: Add more options.,
Maxime Devos <=
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/22
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Ludovic Courtès, 2022/03/22
- [bug#54352] [PATCH] services: dnsmasq: Add more options., Remco van 't Veer, 2022/03/23
[bug#54352] [PATCH v2] services: dnsmasq: Add more options., Maxime Devos, 2022/03/20
[bug#54352] [PATCH v2] services: dnsmasq: Add more options., Maxime Devos, 2022/03/20
[bug#54352] [PATCH v2] services: dnsmasq: Add more options., Maxime Devos, 2022/03/20