Sound good +1 from my side.
> Am 06.07.2018 um 13:10 schrieb Justin Mclean <[email protected]>:
>
> Go for it
>
> On Fri., 6 Jul. 2018, 7:43 pm Christofer Dutz, <[email protected]>
> wrote:
>
>> Ok,
>>
>>
>>
>> I am interpreting the silence as consent and will do the changes I
>> proposed.
>>
>>
>>
>> Chris
>>
>>
>>
>>
>>
>> Am 03.07.18, 13:57 schrieb "Christofer Dutz" <[email protected]>:
>>
>>
>>
>> Hi all,
>>
>>
>>
>> now digging into the subscription features of EtherNet/IP, I also had
>> a first detailed look at the API for Subscriptions in PLC4X.
>>
>>
>>
>> I would like to propose refactoring that a little.
>>
>>
>>
>> I would propose to create SubscriptionRequest and -Response objects
>> containing SubscriptionItems.
>>
>> This way the API would match the Read and Write parts.
>>
>>
>>
>> Above that it would allow subscribing and unsubscribing to multiple
>> addresses in one request.
>>
>> It would also return a response object that can be used to unsubscribe
>> again.
>>
>> The current solution, passing in the same combination of parameters to
>> the unsubscription as used when subscribing, is a little fragile in my
>> opinion.
>>
>>
>>
>> What do you think?
>>
>>
>>
>> Chris
>>
>>
>>
>>
>>
