Re: [tipc-discussion] [PATCH net-next v3 10/12] tipc: make media structure contain tipc_bearer object

[Ying X. <yin...@wi...>]

On 10/07/2013 05:01 PM, Erik Hugne wrote:
> On Mon, Sep 30, 2013 at 04:38:22PM +0800, Ying Xue wrote:
>> In tipc_bearer struct there has usr_handle member pointing to media
>> object; in media struct, there is also a similar pointer called bearer
>> pointing to bearer object. When they access each other, one has to
>> use its pointer to access peer. But as the life cycle of both objects
>> is always same, now we let media struct contain bearer object directly,
>> having their relationship simpler.
>>
>> Signed-off-by: Ying Xue <yin...@wi...>
> 
> I suggest the following:
> 
> The lifecycle of media and bearers are the same, and their relationship 
> can be simplified by directly aggregating the tipc_bearer structure in the 
> tipc_media. This eliminates the need for media/bearer access pointers in
> respective structs.
> 

Thanks, your description is pretty good :)

Regards,
Ying

> //E
>