* [FFmpeg-devel] [PATCH] avutil/parseutils: use quadhd for Quad HD
@ 2022-01-11 2:15 lance.lmwang
2022-01-12 9:49 ` Andreas Rheinhardt
0 siblings, 1 reply; 3+ messages in thread
From: lance.lmwang @ 2022-01-11 2:15 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Limin Wang
From: Limin Wang <lance.lmwang@gmail.com>
qHD is 960x540 (q stands for quarter) and QHD is 2560x1440 (Q is quad).
use quadhd for QHD for abbreviation.
Fix ticket#9591
Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
---
libavutil/parseutils.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
index 19bbdde..94e88e0 100644
--- a/libavutil/parseutils.c
+++ b/libavutil/parseutils.c
@@ -112,7 +112,7 @@ static const VideoSizeAbbr video_size_abbrs[] = {
{ "hd480", 852, 480 },
{ "hd720", 1280, 720 },
{ "hd1080", 1920,1080 },
- { "qhd", 2560,1440 },
+ { "quadhd", 2560,1440 },
{ "2k", 2048,1080 }, /* Digital Cinema System Specification */
{ "2kdci", 2048,1080 },
{ "2kflat", 1998,1080 },
--
1.8.3.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] 3+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avutil/parseutils: use quadhd for Quad HD
2022-01-11 2:15 [FFmpeg-devel] [PATCH] avutil/parseutils: use quadhd for Quad HD lance.lmwang
@ 2022-01-12 9:49 ` Andreas Rheinhardt
2022-01-12 9:52 ` lance.lmwang
0 siblings, 1 reply; 3+ messages in thread
From: Andreas Rheinhardt @ 2022-01-12 9:49 UTC (permalink / raw)
To: ffmpeg-devel
lance.lmwang@gmail.com:
> From: Limin Wang <lance.lmwang@gmail.com>
>
> qHD is 960x540 (q stands for quarter) and QHD is 2560x1440 (Q is quad).
> use quadhd for QHD for abbreviation.
>
> Fix ticket#9591
>
> Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> ---
> libavutil/parseutils.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
> index 19bbdde..94e88e0 100644
> --- a/libavutil/parseutils.c
> +++ b/libavutil/parseutils.c
> @@ -112,7 +112,7 @@ static const VideoSizeAbbr video_size_abbrs[] = {
> { "hd480", 852, 480 },
> { "hd720", 1280, 720 },
> { "hd1080", 1920,1080 },
> - { "qhd", 2560,1440 },
> + { "quadhd", 2560,1440 },
> { "2k", 2048,1080 }, /* Digital Cinema System Specification */
> { "2kdci", 2048,1080 },
> { "2kflat", 1998,1080 },
>
No objections from me for that. If no one else objects, you should apply
it and also backport it to 5.0.
- 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] 3+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avutil/parseutils: use quadhd for Quad HD
2022-01-12 9:49 ` Andreas Rheinhardt
@ 2022-01-12 9:52 ` lance.lmwang
0 siblings, 0 replies; 3+ messages in thread
From: lance.lmwang @ 2022-01-12 9:52 UTC (permalink / raw)
To: ffmpeg-devel
On Wed, Jan 12, 2022 at 10:49:56AM +0100, Andreas Rheinhardt wrote:
> lance.lmwang@gmail.com:
> > From: Limin Wang <lance.lmwang@gmail.com>
> >
> > qHD is 960x540 (q stands for quarter) and QHD is 2560x1440 (Q is quad).
> > use quadhd for QHD for abbreviation.
> >
> > Fix ticket#9591
> >
> > Signed-off-by: Limin Wang <lance.lmwang@gmail.com>
> > ---
> > libavutil/parseutils.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/libavutil/parseutils.c b/libavutil/parseutils.c
> > index 19bbdde..94e88e0 100644
> > --- a/libavutil/parseutils.c
> > +++ b/libavutil/parseutils.c
> > @@ -112,7 +112,7 @@ static const VideoSizeAbbr video_size_abbrs[] = {
> > { "hd480", 852, 480 },
> > { "hd720", 1280, 720 },
> > { "hd1080", 1920,1080 },
> > - { "qhd", 2560,1440 },
> > + { "quadhd", 2560,1440 },
> > { "2k", 2048,1080 }, /* Digital Cinema System Specification */
> > { "2kdci", 2048,1080 },
> > { "2kflat", 1998,1080 },
> >
>
> No objections from me for that. If no one else objects, you should apply
> it and also backport it to 5.0.
Thanks, will apply and backport it to 5.0
>
> - 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] 3+ messages in thread
end of thread, other threads:[~2022-01-12 9:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-11 2:15 [FFmpeg-devel] [PATCH] avutil/parseutils: use quadhd for Quad HD lance.lmwang
2022-01-12 9:49 ` Andreas Rheinhardt
2022-01-12 9:52 ` 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