* Re: [FFmpeg-devel] [PATCH 1/2] avutil/parseutils: add qhd(Quad HD) or wqhd(Wide Quad HD) for 1440p [not found] <1635344836-23293-1-git-send-email-lance.lmwang@gmail.com> @ 2022-01-10 18:16 ` Andreas Rheinhardt 2022-01-11 1:28 ` lance.lmwang 0 siblings, 1 reply; 2+ messages in thread From: Andreas Rheinhardt @ 2022-01-10 18:16 UTC (permalink / raw) To: ffmpeg-devel lance.lmwang@gmail.com: > From: Limin Wang <lance.lmwang@gmail.com> > > Signed-off-by: Limin Wang <lance.lmwang@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? - Andreas _______________________________________________ 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] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH 1/2] avutil/parseutils: add qhd(Quad HD) or wqhd(Wide Quad HD) for 1440p 2022-01-10 18:16 ` [FFmpeg-devel] [PATCH 1/2] avutil/parseutils: add qhd(Quad HD) or wqhd(Wide Quad HD) for 1440p Andreas Rheinhardt @ 2022-01-11 1:28 ` lance.lmwang 0 siblings, 0 replies; 2+ messages in thread From: lance.lmwang @ 2022-01-11 1:28 UTC (permalink / raw) To: ffmpeg-devel On Mon, Jan 10, 2022 at 07:16:50PM +0100, Andreas Rheinhardt wrote: > lance.lmwang@gmail.com: > > From: Limin Wang <lance.lmwang@gmail.com> > > > > Signed-off-by: Limin Wang <lance.lmwang@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@ffmpeg.org > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel > > To unsubscribe, visit link above, or email > ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". -- Thanks, Limin Wang _______________________________________________ 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] 2+ messages in thread
end of thread, other threads:[~2022-01-11 1:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1635344836-23293-1-git-send-email-lance.lmwang@gmail.com>
2022-01-10 18:16 ` [FFmpeg-devel] [PATCH 1/2] avutil/parseutils: add qhd(Quad HD) or wqhd(Wide Quad HD) for 1440p Andreas Rheinhardt
2022-01-11 1:28 ` lance.lmwang
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