[Dnsmasq-discuss] [PATCH] Add new extensible D-Bus signal

Pali Rohár pali.rohar at gmail.com
Thu Jan 9 11:27:42 GMT 2020


On Thursday 09 January 2020 12:17:52 Florent Fourcot wrote:
> Hello,
> 
> On 09/01/2020 12:10, Pali Rohár wrote:
> > Hello! I looked at this API, but is has same problems as old one. It is
> > not possible to extend it in future (if it would be needed). Could you
> > please design API in way that can be extended in future? (See my
> > previous email with details how to achieve it)
> 
> Thanks for our time. Additional data are now in lease_info, a dictionary (as
> you suggested in patch V1). Previous values already in old signals are
> hardcoded, to simplify migration of users from previous to this new one.

Ah, right. So you put there also all previous parameters from old
signal (which are fixed and cannot be changed).

And then new parameters are put into dictionary which is part of those
old parameters.

Well, this can work.

> So, I'm not sure to understand your point. Could you be more specific?

-- 
Pali Rohár
pali.rohar at gmail.com



More information about the Dnsmasq-discuss mailing list