Re: [FFmpeg-devel] [PATCH] configure: Print large lists in more columns if the screen size allows

2015-09-15 Thread Timothy Gu
On Mon, Sep 14, 2015 at 5:09 PM Ganesh Ajjanagadde wrote: > Shouldn't this function be renamed, since it no longer necessarily > prints in 3 columns? > Applied with that change. Timothy ___ ffmpeg-devel mailing list

Re: [FFmpeg-devel] [PATCH] configure: Print large lists in more columns if the screen size allows

2015-09-15 Thread Michael Niedermayer
On Mon, Sep 14, 2015 at 04:46:14PM -0700, Timothy Gu wrote: > --- > configure | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) I have no oppinion on the function name (could be changed, could be a seperate patch) rest LGTM thanks [...] -- Michael GnuPG fingerprint:

[FFmpeg-devel] [PATCH] configure: Print large lists in more columns if the screen size allows

2015-09-14 Thread Timothy Gu
--- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configure b/configure index da18e70..3dfbb91 100755 --- a/configure +++ b/configure @@ -425,7 +425,10 @@ if test -t 1 && which tput >/dev/null; then error_color=$(tput setaf 1)

Re: [FFmpeg-devel] [PATCH] configure: Print large lists in more columns if the screen size allows

2015-09-14 Thread Ganesh Ajjanagadde
On Mon, Sep 14, 2015 at 7:46 PM, Timothy Gu wrote: > --- > configure | 6 +- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index da18e70..3dfbb91 100755 > --- a/configure > +++ b/configure > @@ -425,7 +425,10 @@ if test -t 1