[waffle] [PATCH 02/11] core: Define intptr_t variants of wcore_attrib_list functions

[Emil Velikov]

On 21 December 2014 at 14:22, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> On 16 December 2014 at 08:18, Chad Versace <chad.versace at linux.intel.com> wrote:
>> When designing the original Waffle 1.0 API, I made a mistake when
>> I chose to declare attribute lists as arrays of int32_t. Instead, they
>> should have been arrays of intptr_t.
>>
>> A new public function, waffle_window_create2, will have a `const
>> intptr_t attrib_list[]` parameter. Therefore waffle needs intptr_t
>> variants of the wcore_attrib_list functions.
>>
>> Signed-off-by: Chad Versace <chad.versace at linux.intel.com>
>> ---
>>  src/waffle/core/wcore_attrib_list.c | 73 +++++++++++++++++++++++++++++++++++++
>>  src/waffle/core/wcore_attrib_list.h | 23 ++++++++++++
>>  2 files changed, 96 insertions(+)
>>
>> diff --git a/src/waffle/core/wcore_attrib_list.c b/src/waffle/core/wcore_attrib_list.c
>> index 09a4dec..a7f087d 100644
>> --- a/src/waffle/core/wcore_attrib_list.c
>> +++ b/src/waffle/core/wcore_attrib_list.c
>> @@ -29,6 +29,79 @@
>>  #include <stdint.h>
>>  #include <stddef.h>
>>
>> +size_t
>> +wcore_attrib_list_length(const intptr_t attrib_list[])
>> +{
>> +    const intptr_t *i = attrib_list;
>> +
>> +    if (attrib_list == NULL)
>> +        return 0;
>> +
>> +    while (*i != 0)
>> +        i += 2;
>> +
>> +    return (i - attrib_list) / 2;
> Guessing that 2 is meant sizeof(intptr_t). If so can we use it ?
> Similar request for the other 2's in this patch.
>
Scratch that - seems like I was in the clouds. We're talking about
key/value pairs.

-Emil