[FFmpeg-devel] [PATCH 1/2] avutil/parseutils: add qhd(Quad HD) or wqhd(Wide Quad HD) for 1440p
lance.lmwang at gmail.com
lance.lmwang at gmail.com
Tue Jan 11 03:28:01 EET 2022
On Mon, Jan 10, 2022 at 07:16:50PM +0100, Andreas Rheinhardt wrote:
> lance.lmwang at gmail.com:
> > From: Limin Wang <lance.lmwang at gmail.com>
> >
> > Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> > ---
> > libavutil/parseutils.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
> > index 7f678cd..19bbdde 100644
> > --- a/libavutil/parseutils.c
> > +++ b/libavutil/parseutils.c
> > @@ -102,6 +102,7 @@ static const VideoSizeAbbr video_size_abbrs[] = {
> > { "wsxga", 1600,1024 },
> > { "wuxga", 1920,1200 },
> > { "woxga", 2560,1600 },
> > + { "wqhd", 2560,1440 },
> > { "wqsxga", 3200,2048 },
> > { "wquxga", 3840,2400 },
> > { "whsxga", 6400,4096 },
> > @@ -111,6 +112,7 @@ static const VideoSizeAbbr video_size_abbrs[] = {
> > { "hd480", 852, 480 },
> > { "hd720", 1280, 720 },
> > { "hd1080", 1920,1080 },
> > + { "qhd", 2560,1440 },
> > { "2k", 2048,1080 }, /* Digital Cinema System Specification */
> > { "2kdci", 2048,1080 },
> > { "2kflat", 1998,1080 },
> >
>
> This code added a second qhd abbreviation to this list; it precedes and
> therefore overrides the earlier dimension of 960x540. Ticket #9591 is
> about this.
> According to Wikipedia, qHD is 960x540 (q stands for quarter in this
> case) and QHD is 2560x1440 (Q is quad). So maybe an ad-hoc check for
> this would be appropriate?
Sorry, I'll change the qhd to quadhd to avoid the name conflict.
>
> - Andreas
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
--
Thanks,
Limin Wang
More information about the ffmpeg-devel
mailing list