<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED --- - High latency in HTTP streaming"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=65703#c56">Comment # 56</a>
              on <a class="bz_bug_link 
          bz_status_ASSIGNED "
   title="ASSIGNED --- - High latency in HTTP streaming"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=65703">bug 65703</a>
              from <span class="vcard"><a class="email" href="mailto:tanuk@iki.fi" title="Tanu Kaskinen <tanuk@iki.fi>"> <span class="fn">Tanu Kaskinen</span></a>
</span></b>
        <pre>I very much doubt that adding a rewind callback is the correct fix. The reason
is that nothing in the code base currently implements the source output
process_rewind() callback, so it doesn't seem to be important (I don't fully
understand how source rewinding works - it's quite different from sink
rewinding).

Also, source outputs can be rewound only when connected to a monitor source.
Are you using a monitor source for testing? If not, then the callback is never
called. How can the patch then have any effect on the latency? It seems that
pa_source_output_push() behaves differently if the process_rewind() callback is
set (even if it's never called):

    limit = o->process_rewind ? 0 : o->source->thread_info.max_rewind;</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the QA Contact for the bug.</li>
      </ul>
    </body>
</html>