dlls/cabinet: fix dead stores (llvm/clang)

Dmitry Timoshkov dmitry at codeweavers.com
Tue Nov 18 01:11:52 CST 2008


"ricardo filipe" <ricardo_barbano at hotmail.com> wrote:

> --- a/dlls/cabinet/fci.c
> +++ b/dlls/cabinet/fci.c
> @@ -462,7 +462,7 @@ static cab_ULONG fci_get_checksum(const void *pv, UINT cb, CHECKSUM seed)
>      case 2:
>        ul |= (((ULONG)(*pb++)) <<  8);
>      case 1:
> -      ul |= *pb++;
> +      ul |= *pb;
>      default:
>        break;
>    }

Is there any reason that you ignored 2 similar cases above?

-- 
Dmitry.



More information about the wine-devel mailing list