[FFmpeg-devel] [PATCH 3/4] avcodec/xbmdec: convert() minor speed increase

Jose Da Silva digital at joescat.com
Mon Feb 1 05:51:04 EET 2021


If we test for {0..9} first, we have tested for 10/16th of all possible
characters first and avoid testing the remaining 6/16th of all possible
characters, which can be either 6/16th lowercase or 6/16th uppercase.

Signed-off-by: Joe Da Silva <digital at joescat.com>
---
 libavcodec/xbmdec.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/libavcodec/xbmdec.c b/libavcodec/xbmdec.c
index b783d5abe5..52615dc7ab 100644
--- a/libavcodec/xbmdec.c
+++ b/libavcodec/xbmdec.c
@@ -28,12 +28,12 @@

 static int convert(uint8_t x)
 {
-    if (x >= 'a')
-        x -= 87;
-    else if (x >= 'A')
-        x -= 55;
-    else
+    if (x <= '9')
         x -= '0';
+    else if (x >= 'a')
+        x -= ('a' - 10);
+    else
+        x -= ('A' - 10);
     return x;
 }

--
2.30.0



More information about the ffmpeg-devel mailing list