Re: [PATCH 2/3] client: add helper to call method via builder

[James Prestwood <prestwoj@gmail.com>]

Hi Denis,

On 11/10/23 4:28 AM, James Prestwood wrote:
> Hi Denis,
> 
> On 11/9/23 6:22 PM, Denis Kenzior wrote:
>> Hi James,
>>
>> On 11/9/23 12:49, James Prestwood wrote:
>>> There was no way to call a method with anything except basic
>>> arguments. Add a helper to use a builder to create more complex
>>> args.
>>> ---
>>>   client/dbus-proxy.c | 36 +++++++++++++++++++++++++++++++++++-
>>>   client/dbus-proxy.h |  9 +++++++++
>>>   2 files changed, 44 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/client/dbus-proxy.c b/client/dbus-proxy.c
>>> index 42b8427f..de252427 100644
>>> --- a/client/dbus-proxy.c
>>> +++ b/client/dbus-proxy.c
>>> @@ -2,7 +2,7 @@
>>>    *
>>>    *  Wireless daemon for Linux
>>>    *
>>> - *  Copyright (C) 2017-2020  Intel Corporation. All rights reserved.
>>> + *  Copyright (C) 2017-2023  Intel Corporation. All rights reserved.
>>
>> Is this correct?
> 
> I'm generally terrible about updating the dates on the copyright and 
> really never do it. Decided I'd try and do better this time :)
> 
> But I'm not actually sure how this works. Should there be multiple 
> copyright notices if different companies touch existing files? or should 
> we just leave the copyrights as-is if the file already exists.

Nevermind, took a look at the kernel and they just append the 
companies/dates. It had been so long at Intel I forgot how all this 
worked :)

Thanks,
James