On 05/19/2016 05:48 AM, Petri Hintukainen wrote:
> On ke, 2016-05-18 at 15:03 -0700, John Stebbins wrote:
>> ---
>> src/libbluray/bluray.c | 1 +
>> src/libbluray/bluray.h | 1 +
>> 2 files changed, 2 insertions(+)
>>
>> diff --git a/src/libbluray/bluray.c b/src/libbluray/bluray.c
>> index f9f2502..1910664 100644
>> --- a/src/libbluray/bluray.c
>> +++ b/src/libbluray/bluray.c
>> @@ -2575,6 +2575,7 @@ static BLURAY_TITLE_INFO*
>> _fill_title_info(NAV_TITLE* title, uint32_t title_idx,
>> BLURAY_CLIP_INFO *ci = &title_info->clips[ii];
>> NAV_CLIP *nc = &title->clip_list.clip[ii];
>>
>> + memcpy(ci->clip_id, pi->clip->clip_id, sizeof(ci->clip_id));
>> ci->pkt_count = nc->end_pkt - nc->start_pkt;
>> ci->start_time = (uint64_t)nc->title_time * 2;
>> ci->in_time = (uint64_t)pi->in_time * 2;
>> diff --git a/src/libbluray/bluray.h b/src/libbluray/bluray.h
>> index 41ba978..dc37398 100644
>> --- a/src/libbluray/bluray.h
>> +++ b/src/libbluray/bluray.h
>> @@ -219,6 +219,7 @@ typedef struct bd_stream_info {
>> } BLURAY_STREAM_INFO;
>>
>> typedef struct bd_clip {
>> + char clip_id[6];
> This will break ABI. But, alternative ways of adding this information
> are more or less ugly ...
>
> If we break ABI, there are couple of other structs that could be re-
> arranged too.I have not problem with delaying this till an appropriate time for an ABI break. I would just like to see this eventually be available. Should we create an ABI-break to collect such things in? -- John GnuPG fingerprint: D0EC B3DB C372 D1F1 0B01 83F0 49F1 D7B2 60D4 D0F7
signature.asc
Description: OpenPGP digital signature
_______________________________________________ libbluray-devel mailing list libbluray-devel@videolan.org https://mailman.videolan.org/listinfo/libbluray-devel
