[FFmpeg-devel] [PATCH v2 1/2] Move bitswap_32() into a header file.

James Almer jamrial at gmail.com
Sun Jan 8 16:01:14 EET 2017


On 1/8/2017 10:55 AM, Steinar H. Gunderson wrote:
> On Sun, Jan 08, 2017 at 10:52:52AM -0300, James Almer wrote:
>> Also, when moving code in general we expect a patch that makes use of said
>> code as part of a set.
> 
> I don't understand. I send a patch where I both make the move and add a
> new use. Then I get told to split them into two separate patches and do so;
> now I get told that they need to come together? :-)
> 
> /* Steinar */

My bad, I see now this is patch 1 of 2. I didn't notice the second one
since it's on a separate thread instead of as a reply to this one. Hence
why i mentioned this first patch alone is not of much use.

Disregard that part then. Just address the name of the function.



More information about the ffmpeg-devel mailing list