Просмотр исходного кода

video_core: Enforce -Werror=type-limits

Silences one warning and avoids introducing more in the future.
ReinUsesLisp 5 лет назад
Родитель
Сommit
4a451e5849
2 измененных файлов с 2 добавлено и 1 удалено
  1. 1 0
      src/video_core/CMakeLists.txt
  2. 1 1
      src/video_core/command_classes/codecs/vp9.cpp

+ 1 - 0
src/video_core/CMakeLists.txt

@@ -303,6 +303,7 @@ else()
         -Werror=conversion
         -Werror=conversion
         -Wno-error=sign-conversion
         -Wno-error=sign-conversion
         -Werror=switch
         -Werror=switch
+        -Werror=type-limits
         -Werror=unused-variable
         -Werror=unused-variable
 
 
         $<$<CXX_COMPILER_ID:GNU>:-Werror=class-memaccess>
         $<$<CXX_COMPILER_ID:GNU>:-Werror=class-memaccess>

+ 1 - 1
src/video_core/command_classes/codecs/vp9.cpp

@@ -893,7 +893,7 @@ void VpxRangeEncoder::Write(bool bit, s32 probability) {
         if (((low_value << (offset - 1)) >> 31) != 0) {
         if (((low_value << (offset - 1)) >> 31) != 0) {
             const s32 current_pos = static_cast<s32>(base_stream.GetPosition());
             const s32 current_pos = static_cast<s32>(base_stream.GetPosition());
             base_stream.Seek(-1, Common::SeekOrigin::FromCurrentPos);
             base_stream.Seek(-1, Common::SeekOrigin::FromCurrentPos);
-            while (base_stream.GetPosition() >= 0 && PeekByte() == 0xff) {
+            while (PeekByte() == 0xff) {
                 base_stream.WriteByte(0);
                 base_stream.WriteByte(0);
 
 
                 base_stream.Seek(-2, Common::SeekOrigin::FromCurrentPos);
                 base_stream.Seek(-2, Common::SeekOrigin::FromCurrentPos);