<div dir="ltr">Sorry about the reconstructed email, but I wasn't subscribed to this list beforehand (Thanks Roger for the heads-up)<br><div><div class="gmail_extra"><br>> On 2014-08-15 07:49, Diego Biurrun wrote:<br>
<br>>> Resolve compiler warning (admap_entries should be unsigned int)<br><br>> Hmm, here it is unsigned int ..<br><br>>> -    int admap_entries = (admap->last_byte + 1 - VOBU_ADMAP_SIZE)/VOBU_ADMAP_SIZE;<br>
>> +    uint32_t admap_entries = (admap->last_byte + 1 - VOBU_ADMAP_SIZE)/VOBU_ADMAP_SIZE;<br><br>> .. and here it is uint32_t.<br><br></div><div class="gmail_extra">Yeah, my commit message should have said "Resolve compiler warning (admap_entries should be uint32_t)". My mistake. I don't think there is an easy way to update git commit messages, but let me know if otherwise.<br>
<br></div><div class="gmail_extra">Also, in case you're curious, I chose uint32_t because admap->last_byte is also defined as uint32_t. See snippet below from lines 382-385 in src/dvdread/ifo_types.h. If you feel it should be something else (like unsigned int?), let me know. My main objective was to get rid of the last compiler warning, and the change seemed innocuous.<br>
<br>Thanks.<br><br>typedef struct {<br>  uint32_t last_byte;<br>  uint32_t *vobu_start_sectors;<br>} ATTRIBUTE_PACKED vobu_admap_t;<br><br></div></div></div>