Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion
@ 2024-03-06  0:53 Andreas Rheinhardt
  2024-03-08  0:04 ` Andreas Rheinhardt
  2024-03-09 19:39 ` Andreas Rheinhardt
  0 siblings, 2 replies; 7+ messages in thread
From: Andreas Rheinhardt @ 2024-03-06  0:53 UTC (permalink / raw)
  To: ffmpeg-devel; +Cc: Andreas Rheinhardt

Use a small LUT instead. Improves performance.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/xbmenc.c | 21 +++++++++++++++------
 1 file changed, 15 insertions(+), 6 deletions(-)

diff --git a/libavcodec/xbmenc.c b/libavcodec/xbmenc.c
index cd8b73afa3..5231d4691d 100644
--- a/libavcodec/xbmenc.c
+++ b/libavcodec/xbmenc.c
@@ -20,11 +20,9 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include "libavutil/reverse.h"
 #include "avcodec.h"
 #include "codec_internal.h"
 #include "encode.h"
-#include "mathops.h"
 
 #define ANSI_MIN_READLINE 509
 
@@ -57,14 +55,25 @@ static int xbm_encode_frame(AVCodecContext *avctx, AVPacket *pkt,
     buf += snprintf(buf, 39, "static unsigned char image_bits[] = {\n");
     for (i = 0, l = lineout; i < avctx->height; i++) {
         for (j = 0; j < linesize; j++) {
-            buf += snprintf(buf, 6, " 0x%02X", ff_reverse[*ptr++]);
+            // 0..15 bitreversed as chars
+            static const char lut[] = {
+                '0', '8', '4', 'C', '2', 'A', '6', 'E',
+                '1', '9', '5', 'D', '3', 'B', '7', 'F'
+            };
+            buf[0] = ' ';
+            buf[1] = '0';
+            buf[2] = 'x';
+            buf[3] = lut[*ptr & 0xF];
+            buf[4] = lut[*ptr >> 4];
+            buf += 5;
+            ptr++;
             if (--commas <= 0) {
-                buf += snprintf(buf, 2, "\n");
+                *buf++ = '\n';
                 break;
             }
-            buf += snprintf(buf, 2, ",");
+            *buf++ = ',';
             if (--l <= 0) {
-                buf += snprintf(buf, 2, "\n");
+                *buf++ = '\n';
                 l = lineout;
             }
         }
-- 
2.40.1

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2024-03-09 19:39 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-06  0:53 [FFmpeg-devel] [PATCH] avcodec/xbmenc: Avoid snprintf() for data->hex conversion Andreas Rheinhardt
2024-03-08  0:04 ` Andreas Rheinhardt
2024-03-08  0:51   ` Marton Balint
2024-03-08  1:04     ` Andreas Rheinhardt
2024-03-08  1:26       ` Marton Balint
2024-03-08 12:17         ` Andreas Rheinhardt
2024-03-09 19:39 ` Andreas Rheinhardt

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git