virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2024-12-23 12:43:46 +02:00
Code Issues Releases Activity

lavu: Remove bit packing from AVComponentDescriptor

Browse Source 
There is no practical benefit in having this structure elements
bit packed given the size of the structure and its usage.
Change types from uint16_t (packed) to plain int in order to simplify
modifying the structure and accessing its fields.

Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
This commit is contained in:
Vittorio Giovara 2015-09-02 18:59:38 +02:00
parent b8b5d82744
commit 6b3ef7f080
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
doc/APIchanges
View File

@ -15,6 +15,8 @@ API changes, most recent first:
2015-xx-xx - lavu 55.0.0 2015-xx-xx - lavu 55.0.0
xxxxxxx - Change type of AVPixFmtDescriptor.flags from uint8_t to uint64_t. xxxxxxx - Change type of AVPixFmtDescriptor.flags from uint8_t to uint64_t.
xxxxxxx - Change type of AVComponentDescriptor fields from uint16_t to int
and drop bit packing.
2015-xx-xx - lavu 54.17.0 2015-xx-xx - lavu 54.17.0
xxxxxxx - Add av_blowfish_alloc(). xxxxxxx - Add av_blowfish_alloc().

10
libavutil/pixdesc.h
View File

@ -31,30 +31,30 @@ typedef struct AVComponentDescriptor {
/** /**
* Which of the 4 planes contains the component. * Which of the 4 planes contains the component.
*/ */
uint16_t plane : 2; int plane;
/** /**
* Number of elements between 2 horizontally consecutive pixels minus 1. * Number of elements between 2 horizontally consecutive pixels minus 1.
* Elements are bits for bitstream formats, bytes otherwise. * Elements are bits for bitstream formats, bytes otherwise.
*/ */
uint16_t step_minus1 : 3; int step_minus1;
/** /**
* Number of elements before the component of the first pixel plus 1. * Number of elements before the component of the first pixel plus 1.
* Elements are bits for bitstream formats, bytes otherwise. * Elements are bits for bitstream formats, bytes otherwise.
*/ */
uint16_t offset_plus1 : 3; int offset_plus1;
/** /**
* Number of least significant bits that must be shifted away * Number of least significant bits that must be shifted away
* to get the value. * to get the value.
*/ */
uint16_t shift : 3; int shift;
/** /**
* Number of bits in the component minus 1. * Number of bits in the component minus 1.
*/ */
uint16_t depth_minus1 : 4; int depth_minus1;
} AVComponentDescriptor; } AVComponentDescriptor;
/** /**