Package: libjpeg-turbo
Version: 1:1.3.1-11
Severity: minor
Tags: upstream

It would be nice to rewrite the following piece of code:

static const int extend_offset[16] = /* entry n is (-1 << n) + 1 */
  { 0, ((-1)<<1) + 1, ((-1)<<2) + 1, ((-1)<<3) + 1, ((-1)<<4) + 1,
    ((-1)<<5) + 1, ((-1)<<6) + 1, ((-1)<<7) + 1, ((-1)<<8) + 1,
    ((-1)<<9) + 1, ((-1)<<10) + 1, ((-1)<<11) + 1, ((-1)<<12) + 1,
    ((-1)<<13) + 1, ((-1)<<14) + 1, ((-1)<<15) + 1 };

Simply moving the negative sign outside the parenthesis should be
fine: ((-1)<<15) + 1 == -(1<<15) + 1

Reply via email to