[x265] Warnings in GCC 4.8.2 for Win32: Comp. signed/unsigned

Steve Borho steve at borho.org
Wed Jan 14 12:21:47 CET 2015


On 01/14, Mario *LigH* Rohkr??mer wrote:
> Warnings in compiles with Win32 target only:
> 
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp: In function
> 'uint64_t computeSSD(pixel*, pixel*, intptr_t, uint32_t, uint32_t)':
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:328:59:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>      for (int size = BLOCK_64x64; size >= BLOCK_4x4 && y < height; size--)
>                                                            ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:332:33:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>          for (; y + rowHeight <= height; y += rowHeight)
>                                  ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:338:34:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>                  for (; x + 64 <= width; x += 64)
>                                   ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:342:34:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>                  for (; x + 32 <= width; x += 32)
>                                   ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:347:34:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>                  for (; x + 16 <= width; x += 16)
>                                   ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:352:33:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>                  for (; x + 8 <= width; x += 8)
>                                  ^
> h:/MSYS/home/Entwicklung/x265/source/encoder/framefilter.cpp:356:29:
> warning: comparison between signed and unsigned integer expressions
> [-Wsign-compare]
>              for (; x + 4 <= width; x += 4)

Thanks, I'll take care of this

-- 
Steve Borho


More information about the x265-devel mailing list