[FFmpeg-devel] [PATCH 1/4] avutil/mips: Restructure of generic macros

Michael Niedermayer michaelni at gmx.at
Tue May 26 14:28:40 CEST 2015


On Tue, May 26, 2015 at 04:49:13PM +0530, shivraj.patil at imgtec.com wrote:
> From: Shivraj Patil <shivraj.patil at imgtec.com>
> 
> Recently we have done little restructuring/styling changes to the optimization sources at our end (like generic macro definitions, their use to reduce code lines, better code alignments etc). 
> This change was also necessary to avoid repeated review comments in remaining patches which we were in process to submit.
> The above restructuring impacted the already submitted code so we will need to re-submit the accepted code again.
> 
> We intend to do this as following
> PATCH 1/4:- Will modify avutil/mips/generic_macros.h
> PATCH 2/4:- Will modify avcodec/mips/h264dsp_msa.c
> PATCH 3/4:- Will disable hevc uni mc for time being (will add it in subsequent patches)
> PATCH 4/4:- Will modify avcodec/mips/hevcdsp_msa.c
> 
> Overall, the above patches are just upgrading the already accepted code with styling changes and will bring it in sync with our modified codebase.

the above describes all patches, but we need a commit message
for each patch
also messages like "Will modify somefile" are not ok, every commit
will modify something, thats too generic and has no real informative
content about what is actually done

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No great genius has ever existed without some touch of madness. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150526/9d6f8f19/attachment.asc>


More information about the ffmpeg-devel mailing list