Re: [PATCH 0/1] MR7508: include: Add mpeg2data.idl.

[Jacek Caban (＠jacek)]

Jacek Caban (@jacek) commented about include/mpeg2bits.h:
> -
> - This library is distributed in the hope that it will be useful,
> -
> - but WITHOUT ANY WARRANTY; without even the implied warranty of
> -
> - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
> -
> - Lesser General Public License for more details.
> -
> -
> -
> - You should have received a copy of the GNU Lesser General Public
> -
> - License along with this library; if not, write to the Free Software
> -
> - Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
> - */
> -
>
> +#ifndef _MPEG2BITS_H_
> +#define _MPEG2BITS_H_
> -
>
> +#pragma pack(push)
> -
>
> +#pragma pack(1)

Please include `pshpack1.h` instead (and `poppack.h` instead of `pack(pop)`).