<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [PATCH] remove unused variables"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101811#c3">Comment # 3</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [PATCH] remove unused variables"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101811">bug 101811</a>
              from <span class="vcard"><a class="email" href="mailto:aacid@kde.org" title="Albert Astals Cid <aacid@kde.org>"> <span class="fn">Albert Astals Cid</span></a>
</span></b>
        <pre>(In reply to Jannick from <a href="show_bug.cgi?id=101811#c2">comment #2</a>)
<span class="quote">> The clean-up is meant to pacify the compiler complaining about unused or
> even uninitialized variables in the code given the configuration.

> Just to understand what you are saying and to be productive: Where exactly
> breaks your code, since mine works well under my configuration?</span >

You removed the calls to read16 so the stream is not advanced and thus any
subsequent read is wrong. 

<span class="quote">> 
> Probably a good idea to make the *compiler* throw either an information or
> an error that the internal DCTStream is unmaintained and have the dev use
> the alternative code.</span >

Which version are you using? You have to explicitly say you want the
unmaintained code on configure / cmake time.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>