<div class="gmail_quote">On Wed, Feb 24, 2010 at 2:06 PM, Fabio Ritrovato <span dir="ltr"><<a href="mailto:exsephiroth87@gmail.com">exsephiroth87@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Wed, Feb 24, 2010 at 1:54 PM, Pierre d'Herbemont <<a href="mailto:pdherbemont@free.fr">pdherbemont@free.fr</a>> wrote:<br>
><br>
> On Wed, Feb 24, 2010 at 10:00 AM, Jakob Leben <<a href="mailto:jakob.leben@gmail.com">jakob.leben@gmail.com</a>> wrote:<br>
</div><div class="im">>> Hi Fabio, here is my opinions on the patch:<br>
>><br>
>> I believe the services_discovery_t: b_loading field should be protected by<br>
>> some mutex, which services_discovery_t does not have yet.<br>
>><br>
>> I would suggest to:<br>
>><br>
>> A. Either move the b_loading field into playlist's struct<br>
>> vlc_sd_internal_t and use playlist lock to protect it, while adding a public<br>
>> function playlist_IsServicesDiscoveryLoading( ).<br>
><br>
> Was going to suggest the same thing. Except, put it in the (just added)<br>
> services_discovery's private member. And use services_discovery_IsLoading().<br>
>><br>
>> B: Or remove the b_loading field and pass the loading state as bool<br>
>> through callbacks all the way until Qt intf.<br>
><br>
> As you want :)<br>
<br>
</div>I had the feeling something was missing...<br>
<br>
Passing just the state I feel won't give enough informations to the<br>
interface, so I think I'll keep passing the p_sd...<br></blockquote><div><br><br>I agree, I'd prefer storing the state somewhere. I am only undecided as to storing it in vlc_sd_internal_t and using playlist lock, or adding a lock to services_discovery_t. I am inclining towards the latter, as it can be necessary in future anyway.<br>
</div></div>