[FFmpeg-cvslog] libavcodec/jpeg2000_parser: Rearrange ifs

Tomas Härdin git at videolan.org
Fri Jun 10 12:11:59 EEST 2022


ffmpeg | branch: master | Tomas Härdin <git at haerdin.se> | Fri May 20 14:44:54 2022 +0200| [d6d7b78c4609c54966ba06a4dcb93086b9001c22] | committer: Tomas Härdin

libavcodec/jpeg2000_parser: Rearrange ifs

A modest 8% improvement

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=d6d7b78c4609c54966ba06a4dcb93086b9001c22
---

 libavcodec/jpeg2000_parser.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/libavcodec/jpeg2000_parser.c b/libavcodec/jpeg2000_parser.c
index 4cb7db4225..b2ad9f2579 100644
--- a/libavcodec/jpeg2000_parser.c
+++ b/libavcodec/jpeg2000_parser.c
@@ -160,9 +160,10 @@ static int find_frame_end(JPEG2000ParserContext *m, const uint8_t *buf, int buf_
                 return i + 1; // End of frame detected, return frame size.
             }
             m->in_codestream = 0;
-        } else if (m->in_codestream && (state64 & 0xFFFF) == 0xFF90) { // Are we in tile part header?
+        } else if (m->in_codestream) {
+          if ((state64 & 0xFFFF) == 0xFF90) { // Are we in tile part header?
             m->read_tp = 8;
-        } else if (pc->frame_start_found && info_marker((state64 & 0xFFFF0000)>>16) && m->in_codestream && (state64 & 0xFFFF)) {
+          } else if (info_marker((state64 & 0xFFFF0000)>>16) && pc->frame_start_found && (state64 & 0xFFFF)) {
             // Calculate number of bytes to skip to get to end of the next marker.
             m->skip_bytes = (state64 & 0xFFFF)-1;
 
@@ -174,6 +175,7 @@ static int find_frame_end(JPEG2000ParserContext *m, const uint8_t *buf, int buf_
                     m->skip_bytes += 2;
                 }
             }
+          }
         }
     }
 



More information about the ffmpeg-cvslog mailing list