Re: g_value_peek_pointer weirdness



On Tue, 16 Dec 2003, Sheldon Simms wrote:

> So, I guess no one disables checks and then calls
> g_value_peek_pointer() with
> a bad value.
>
> It seems to me that it would be correct to use
> g_assert() here:
>
>   if (!value_table->value_peek_pointer)
>     {
>       g_assert(g_value_fits_pointer(value) == TRUE);
>       return NULL;
>     }
>
>   return value_table->value_peek_pointer(value);

that'd be marginally better, and i wouldn't object to
someone making that change.

>
> -Sheldon
>

---
ciaoTJ




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]