Fixes warnings.

aous72 · Oct 29, 2024 · 5115fb5 · 5115fb5
1 parent 7e5e240
commit 5115fb5
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/src/core/codestream/ojph_tile.cpp b/src/core/codestream/ojph_tile.cpp
@@ -391,7 +391,6 @@ namespace ojph {
             src_line = lines + comp_num;
           else
             src_line = comps[comp_num].pull_line();
-          si32* dp = tgt_line->i32 + line_offsets[comp_num];
           if (is_signed[comp_num] && nlt_type3[comp_num])
             rev_convert_nlt_type3(src_line, 0, tgt_line, 
               line_offsets[comp_num], shift + 1, comp_width);

diff --git a/src/core/transform/ojph_colour.cpp b/src/core/transform/ojph_colour.cpp
@@ -217,9 +217,9 @@ namespace ojph {
       line_buf *dst_line, const ui32 dst_line_offset, 
       si64 shift, ui32 width)
     {
-      if (src_line->flags | line_buf::LFT_32BIT)
+      if ((src_line->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
       { 
-        if (dst_line->flags | line_buf::LFT_32BIT)
+        if ((dst_line->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
         {
           const si32 *sp = src_line->i32 + src_line_offset;
           si32 *dp = dst_line->i32 + dst_line_offset;
@@ -252,9 +252,9 @@ namespace ojph {
       line_buf *dst_line, const ui32 dst_line_offset, 
       si64 shift, ui32 width)
     {
-      if (src_line->flags | line_buf::LFT_32BIT)
+      if ((src_line->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
       { 
-        if (dst_line->flags | line_buf::LFT_32BIT)
+        if ((dst_line->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
         {
           const si32 *sp = src_line->i32 + src_line_offset;
           si32 *dp = dst_line->i32 + dst_line_offset;
@@ -331,7 +331,7 @@ namespace ojph {
              (g->flags  | line_buf::LFT_REVERSIBLE) && 
              (b->flags  | line_buf::LFT_REVERSIBLE));
 
-      if (y->flags | line_buf::LFT_32BIT)
+      if  ((y->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
       {
         assert((y->flags  | line_buf::LFT_32BIT) &&
                (cb->flags | line_buf::LFT_32BIT) && 
@@ -381,7 +381,7 @@ namespace ojph {
              (g->flags  | line_buf::LFT_REVERSIBLE) && 
              (b->flags  | line_buf::LFT_REVERSIBLE));
 
-      if (y->flags | line_buf::LFT_32BIT)
+      if ((y->flags | line_buf::LFT_32BIT) == line_buf::LFT_32BIT)
       {
         assert((y->flags  | line_buf::LFT_32BIT) &&
                (cb->flags | line_buf::LFT_32BIT) &&