tree 6547f4edecc4d15951062343185432bc82df69cb
parent b1398cba43cab0acbb02dcde57c27b3c3f3dc3e3
author DRC <dcommander@users.sourceforge.net> 1397979393 +0000
committer DRC <dcommander@users.sourceforge.net> 1397979393 +0000

Work around an issue in Visual C++ 2010 and 2013 that was causing the 256-bit bitmap test in the regression tests to fail.  More specifically, when optimization is enabled in these versions of Visual C++, the optimizer seems to get confused by the following code block:

       delta = cur0 * 2;
       cur0 += delta;          /* form error * 3 */
       errorptr[0] = (FSERROR) (bpreverr0 + cur0);
       cur0 += delta;          /* form error * 5 */
       bpreverr0 = belowerr0 + cur0;
       cur0 += delta;          /* form error * 7 */

Each time cur0 is incremented by delta, the compiled code doubles the value of delta (WTF?!)  Thus, by the time the end of the block is reached, cur0 is equal to 15 times its former self, not 7 times its former self as it should be.  At any rate, it was a lot simpler to just refactor the code so that it uses multiplication.


git-svn-id: svn+ssh://svn.code.sf.net/p/libjpeg-turbo/code/branches/1.2.x@1255 632fc199-4ca6-4c93-a231-07263d6284db