> On May 22, 2023, at 19:03, Lance Wang <lance.lmw...@gmail.com> wrote:
>
> On Mon, May 22, 2023 at 12:05 PM Zhao Zhili <quinkbl...@foxmail.com> wrote:
>
>> From: Zhao Zhili <zhiliz...@tencent.com>
>>
>> So CJK can be wrapped automatically.
>>
>> Signed-off-by: Zhao Zhili <zhiliz...@tencent.com>
>> ---
>> v2: Don't overwrite wrap automatically for native ASS
>>
>> libavfilter/version.h | 2 +-
>> libavfilter/vf_subtitles.c | 14 ++++++++++++++
>> 2 files changed, 15 insertions(+), 1 deletion(-)
>>
>> diff --git a/libavfilter/version.h b/libavfilter/version.h
>> index ba8a6fdab2..08779130f5 100644
>> --- a/libavfilter/version.h
>> +++ b/libavfilter/version.h
>> @@ -32,7 +32,7 @@
>> #include "version_major.h"
>>
>> #define LIBAVFILTER_VERSION_MINOR 8
>> -#define LIBAVFILTER_VERSION_MICRO 100
>> +#define LIBAVFILTER_VERSION_MICRO 101
>>
>>
>> #define LIBAVFILTER_VERSION_INT AV_VERSION_INT(LIBAVFILTER_VERSION_MAJOR,
>> \
>> diff --git a/libavfilter/vf_subtitles.c b/libavfilter/vf_subtitles.c
>> index 82e140e986..48038cde23 100644
>> --- a/libavfilter/vf_subtitles.c
>> +++ b/libavfilter/vf_subtitles.c
>> @@ -61,6 +61,7 @@ typedef struct AssContext {
>> int original_w, original_h;
>> int shaping;
>> FFDrawContext draw;
>> + int wrap_unicode;
>> } AssContext;
>>
>> #define OFFSET(x) offsetof(AssContext, x)
>> @@ -271,6 +272,9 @@ static const AVOption subtitles_options[] = {
>> {"stream_index", "set stream index",
>> OFFSET(stream_index), AV_OPT_TYPE_INT, { .i64 = -1 }, -1,
>> INT_MAX, FLAGS},
>> {"si", "set stream index",
>> OFFSET(stream_index), AV_OPT_TYPE_INT, { .i64 = -1 }, -1,
>> INT_MAX, FLAGS},
>> {"force_style", "force subtitle style",
>> OFFSET(force_style), AV_OPT_TYPE_STRING, {.str = NULL}, 0, 0, FLAGS},
>> +#if (LIBASS_VERSION >= 0x01600010)
>> + {"wrap_unicode", "break lines according to the Unicode Line Breaking
>> Algorithm", OFFSET(wrap_unicode), AV_OPT_TYPE_BOOL, { .i64 = -1 }, -1, 1,
>> FLAGS },
>> +#endif
>
> {NULL},
>> };
>>
>> @@ -432,6 +436,16 @@ static av_cold int init_subtitles(AVFilterContext
>> *ctx)
>> if (ret < 0)
>> goto end;
>>
>> + /* Don't overwrite wrap automatically for native ASS */
>> + if (ass->wrap_unicode == -1)
>> + ass->wrap_unicode = st->codecpar->codec_id != AV_CODEC_ID_ASS;
>> + if (ass->wrap_unicode) {
>> + ret = ass_track_set_feature(ass->track, ASS_FEATURE_WRAP_UNICODE,
>> 1);
>>
>
> ASS_FEATURE_WRAP_UNICODE is defined if LIBASS_VERSION < 0x01600010?
Oops, I should just remove the #else branch but keep the #if in patch v1.
Will fixed it in v3.
>
>
>> + if (ret < 0)
>> + av_log(ctx, AV_LOG_WARNING,
>> + "libass wasn't built with ASS_FEATURE_WRAP_UNICODE
>> support\n");
>> + }
>> +
>> if (ass->force_style) {
>> char **list = NULL;
>> char *temp = NULL;
>> --
>> 2.25.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-requ...@ffmpeg.org with subject "unsubscribe".
>>
> _______________________________________________
> 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".
_______________________________________________
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".
