<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 15, 2015 at 2:12 PM, Victor Toso <span dir="ltr"><<a href="mailto:victortoso@redhat.com" target="_blank">victortoso@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="">> > +<br>
> > +    if (info->channel_map.channels == 0) {<br>
> > +        SPICE_DEBUG("%s - Number of channels stored is zero. Ignore.<br>
> > (%s)", __func__, info->name);<br>
> > +        return;<br>
> > +<br>
><br>
> What if there is already a playing stream, is this going to reset it to<br>
> restore value?<br>
<br>
</span>Sorry, I think I did not understand.<br>
<br>
This if () is only to avoid issues with restore-info. If the number of<br>
channels stored in pulseaudio database is zero, probably we are not<br>
interested in the volume-information.<br>
 <br></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I'm not sure when I hit this issue, probably when you have a new<br>
database and we start the pulseaudio context but do not have any audio<br>
stream. I believe it saved the application information in the database,<br>
with default values but I'm not sure.<br>
<span class=""></span></blockquote></div><br>It's not about that if, it's actually about the function and the code after that change pstream->info. If there is already a info_cb that did update info, you'll reset it here.<br clear="all"><br>-- <br><div class="gmail_signature">Marc-André Lureau</div>
</div></div>