<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, May 17, 2013 at 8:13 AM, Rafaël Carré <span dir="ltr"><<a href="mailto:funman@videolan.org" target="_blank">funman@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 17/05/2013 03:48, Julien 'Lta' BALLET a écrit :<br>
<div><div class="h5">> Hi,<br>
><br>
> Thank you again funman for reviewing the patch.<br>
> I still have a question.<br>
><br>
><br>
> On Thu, May 16, 2013 at 4:50 PM, Rafaël Carré <<a href="mailto:funman@videolan.org">funman@videolan.org</a>> wrote:<br>
><br>
>>> +// Frame pool for QuickSync video encoder with Intel Media SDK's format frames.<br>
>>> +typedef struct qsv_frame_pool_t<br>
>>> +{<br>
>>> +    mfxFrameInfo          fmt;            // IntelMediaSDK format info.<br>
>>> +    mfxFrameSurface1      *frames;        // An allocated array of 'size' frames.<br>
>>> +    unsigned int          size;           // The number of frame in the pool.<br>
>>> +} qsv_frame_pool_t;<br>
>><br>
>> Still unnecessary padding.<br>
><br>
> Are you talking of the padding at the end of the structure or the<br>
> possible padding between fmt and frames ?<br>
<br>
</div></div>Padding between fmt and frames, the biggest members (pointers) should<br>
appear first in the struct<br>
<div class="HOEnZb"><div class="h5"><br></div></div></blockquote><div><br></div><div style>fmt is an Intel Media SDK defined struct, it's far larger than a pointer. </div><div><br></div></div><br></div></div>