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
ffmpeg-devel@ffmpeg.org
http://ffmp
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: 9FF2128B147
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 && which tput >/dev/null
---
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)
reset_color=$(tp