<div dir="ltr">On Sat, Jun 27, 2015 at 5:08 AM, Erik Faye-Lund <span dir="ltr"><<a href="mailto:kusmabite@gmail.com" target="_blank">kusmabite@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, Jun 26, 2015 at 9:05 PM, Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>> wrote:<br>
> On 06/26/2015 12:06 PM, Erik Faye-Lund wrote:<br>
>><br>
>> In order to save a small leak if mesa is continously loaded and<br>
>> unloaded, let's free the locale when the shared object is unloaded.<br>
>><br>
>> Signed-off-by: Erik Faye-Lund <<a href="mailto:kusmabite@gmail.com">kusmabite@gmail.com</a>><br>
>> Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
>> ---<br>
>>   src/mesa/main/context.c | 12 +++++++++++-<br>
>>   src/util/strtod.c       |  8 ++++++++<br>
>>   src/util/strtod.h       |  3 +++<br>
>>   3 files changed, 22 insertions(+), 1 deletion(-)<br>
>><br>
>> diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c<br>
>> index e68de68..dee1fa8 100644<br>
>> --- a/src/mesa/main/context.c<br>
>> +++ b/src/mesa/main/context.c<br>
>> @@ -346,6 +346,16 @@ _mesa_destroy_visual( struct gl_config *vis )<br>
>>   mtx_t OneTimeLock = _MTX_INITIALIZER_NP;<br>
>><br>
>><br>
>> +/**<br>
>> + * Calls all the various one-time-fini functions in Mesa<br>
>> + */<br>
>> +<br>
>> +static void<br>
>> +one_time_fini()<br>
><br>
><br>
> I think that should be one_time_fini(void) to be consistent and to avoid<br>
> warnings with some compilers.<br>
<br>
Good eyes, I've fixed that locally. Thanks :)<br>
<br>
> Otherwise, the series looks good to me.  Nice to see patch 1.  I remember<br>
> writing that dumb code years ago.<br>
><br>
> Reviewed-by: Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>><br>
<br>
Thanks. Is that Reviewed-by for this patch, or the whole series?<br></blockquote><div><br></div><div>Whole series.<br><br></div><div>-Brian <br></div></div><br></div></div>