On Fri, Jan 11, 2013 at 04:09:02PM +0200, Martin Storsjö wrote:
> Remove leftover debug comments, fix brace placement and
> add whitespace, remove unnecessary and weirdly placed braces.
> 
> Intentionally not adding extra spacing within the braces in the
> table, to avoid extra churn, since it is pretty well formatted
> already.
> ---
>  libavformat/rtp.c |   14 ++++----------
>  1 file changed, 4 insertions(+), 10 deletions(-)

The second paragraph is more of an annotation, IMO leave it out when
you push.  I'd add the spaces myself, but I'll leave it up to you.
Push whenever you want.

> --- a/libavformat/rtp.c
> +++ b/libavformat/rtp.c
> @@ -24,24 +24,20 @@
>  
> -//#define DEBUG
> -

We have more of those pointless things, maybe we should get rid of
them all in one go.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to