[07/15] vc1dec: Fix doxy for vc1_mc_4mv_chroma4()

Message ID 1369729571-75967-7-git-send-email-martin@martin.st
State Committed
Commit c69765a2ccc66bd41f03470ba34899a16de9e2cb
Headers show

Commit Message

Martin Storsjö May 28, 2013, 8:26 a.m.
From: Michael Niedermayer <michaelni@gmx.at>

---
 libavcodec/vc1dec.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kostya Shishkov May 28, 2013, 9:31 a.m. | #1
On Tue, May 28, 2013 at 11:26:03AM +0300, Martin Storsjö wrote:
> From: Michael Niedermayer <michaelni@gmx.at>
> 
> ---
>  libavcodec/vc1dec.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c
> index fb0c5c2..4f4ce5b 100644
> --- a/libavcodec/vc1dec.c
> +++ b/libavcodec/vc1dec.c
> @@ -946,7 +946,7 @@ static void vc1_mc_4mv_chroma(VC1Context *v, int dir)
>      }
>  }
>  
> -/** Do motion compensation for 4-MV field chroma macroblock (both U and V)
> +/** Do motion compensation for 4-MV interlaced frame chroma macroblock (both U and V)
>   */
>  static void vc1_mc_4mv_chroma4(VC1Context *v)
>  {
> -- 

looks OK
Diego Biurrun May 28, 2013, 11:03 a.m. | #2
On 2013-05-28 10:26, Martin Storsjö wrote:
> --- a/libavcodec/vc1dec.c
> +++ b/libavcodec/vc1dec.c
> @@ -946,7 +946,7 @@ static void vc1_mc_4mv_chroma(VC1Context *v, int dir)
>
> -/** Do motion compensation for 4-MV field chroma macroblock (both U and V)
> +/** Do motion compensation for 4-MV interlaced frame chroma macroblock (both U and V)
>    */
>   static void vc1_mc_4mv_chroma4(VC1Context *v)

I don't see why this should be a Doxygen comment, so maybe make that a 
normal comment while you're at it.

Diego
Martin Storsjö May 28, 2013, 11:11 a.m. | #3
On Tue, 28 May 2013, Diego Biurrun wrote:

> On 2013-05-28 10:26, Martin Storsjö wrote:
>> --- a/libavcodec/vc1dec.c
>> +++ b/libavcodec/vc1dec.c
>> @@ -946,7 +946,7 @@ static void vc1_mc_4mv_chroma(VC1Context *v, int dir)
>> 
>> -/** Do motion compensation for 4-MV field chroma macroblock (both U and V)
>> +/** Do motion compensation for 4-MV interlaced frame chroma macroblock 
>> (both U and V)
>>    */
>>   static void vc1_mc_4mv_chroma4(VC1Context *v)
>
> I don't see why this should be a Doxygen comment, so maybe make that a normal 
> comment while you're at it.

Sorry, I pushed the set already when Kostya was ok with it. There's a lot 
of doxygen comments in these files that could be changed all at once 
though, if you feel like it.

// Martin

Patch

diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c
index fb0c5c2..4f4ce5b 100644
--- a/libavcodec/vc1dec.c
+++ b/libavcodec/vc1dec.c
@@ -946,7 +946,7 @@  static void vc1_mc_4mv_chroma(VC1Context *v, int dir)
     }
 }
 
-/** Do motion compensation for 4-MV field chroma macroblock (both U and V)
+/** Do motion compensation for 4-MV interlaced frame chroma macroblock (both U and V)
  */
 static void vc1_mc_4mv_chroma4(VC1Context *v)
 {