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 <digi...@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
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
