From: Manuel Lauss <manuel.lauss@gmail.com>
To: ffmpeg-devel@ffmpeg.org
Cc: Manuel Lauss <manuel.lauss@gmail.com>
Subject: [FFmpeg-devel] [PATCH 1/2] avcodec/sanm: fix codec33/34 tile generator
Date: Fri, 30 May 2025 13:19:02 +0200
Message-ID: <20250530111903.51777-2-manuel.lauss@gmail.com> (raw)
In-Reply-To: <20250530111903.51777-1-manuel.lauss@gmail.com>
tested with LVL11/L11PLAY.ANM from Rebel Assault Sega-CD release.
---
libavcodec/sanm.c | 33 +++++++++++++++------------------
1 file changed, 15 insertions(+), 18 deletions(-)
diff --git a/libavcodec/sanm.c b/libavcodec/sanm.c
index 00bfef00fe..46cad3c703 100644
--- a/libavcodec/sanm.c
+++ b/libavcodec/sanm.c
@@ -493,14 +493,12 @@ static void codec33_gen_tiles(SANMVideoContext *ctx, int8_t param1)
for (k = 0; k < 8; k++) {
j = i + param1;
l = k + param1;
- p = (j + k) / 2;
- j = (j + p) / 2;
- m = l / 2;
- n = (i + param1);
- o = (k + param1);
-
- *dst++ = p; *dst++ = p; *dst++ = j; *dst++ = n;
- *dst++ = p; *dst++ = p; *dst++ = j; *dst++ = i;
+ p = (j + l) >> 1;
+ n = (j + p) >> 1;
+ m = (p + l) >> 1;
+
+ *dst++ = p; *dst++ = p; *dst++ = n; *dst++ = j;
+ *dst++ = p; *dst++ = p; *dst++ = n; *dst++ = j;
*dst++ = m; *dst++ = m; *dst++ = p; *dst++ = j;
*dst++ = l; *dst++ = l; *dst++ = m; *dst++ = p;
}
@@ -510,8 +508,8 @@ static void codec33_gen_tiles(SANMVideoContext *ctx, int8_t param1)
for (k = 0; k < 8; k++) {
j = i + param1;
l = k + param1;
- n = ((j + l) / 2);
- m = ((l + n) / 2);
+ n = (j + l) >> 1;
+ m = (l + n) >> 1;
*dst++ = j; *dst++ = j; *dst++ = j; *dst++ = j;
*dst++ = n; *dst++ = n; *dst++ = n; *dst++ = n;
@@ -524,13 +522,12 @@ static void codec33_gen_tiles(SANMVideoContext *ctx, int8_t param1)
for (k = 0; k < 8; k++) {
j = i + param1;
l = k + param1;
- m = (j + l) / 2;
- n = (j + m) / 2;
- o = m / 2;
- p = j & 0xff;
+ m = (j + l) >> 1;
+ n = (j + m) >> 1;
+ o = (l + m) >> 1;
- *dst++ = p; *dst++ = p; *dst++ = n; *dst++ = m;
- *dst++ = p; *dst++ = p; *dst++ = n; *dst++ = m;
+ *dst++ = j; *dst++ = j; *dst++ = n; *dst++ = m;
+ *dst++ = j; *dst++ = j; *dst++ = n; *dst++ = m;
*dst++ = n; *dst++ = n; *dst++ = m; *dst++ = o;
*dst++ = m; *dst++ = m; *dst++ = o; *dst++ = l;
}
@@ -540,8 +537,8 @@ static void codec33_gen_tiles(SANMVideoContext *ctx, int8_t param1)
for (k = 0; k < 8; k++) {
j = i + param1;
l = k + param1;
- m = (j + l) / 2;
- n = m / 2;
+ m = (j + l) >> 1;
+ n = (l + m) >> 1;
*dst++ = j; *dst++ = m; *dst++ = n; *dst++ = l;
*dst++ = j; *dst++ = m; *dst++ = n; *dst++ = l;
--
2.49.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-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2025-05-30 11:19 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-30 11:19 [FFmpeg-devel] [PATCH 0/2] avcodec/sanm: SEGA-CD codecs support Manuel Lauss
2025-05-30 11:19 ` Manuel Lauss [this message]
2025-05-30 11:19 ` [FFmpeg-devel] [PATCH 2/2] avcodec/sanm: codec31/32 (SEGA-CD) support Manuel Lauss
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250530111903.51777-2-manuel.lauss@gmail.com \
--to=manuel.lauss@gmail.com \
--cc=ffmpeg-devel@ffmpeg.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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