Outside the scope of this PR.  If someone wants to do a cosmetics PR
for it, that'd be fine.  I always cringe at "type* sym = 0;"

On 5/6/14, Rainer Hochecker <notifications@github.com> wrote:
>> @@ -177,11 +177,8 @@ CWinEventsX11Imp::CWinEventsX11Imp()
>>
>>  CWinEventsX11Imp::~CWinEventsX11Imp()
>>  {
>> -  if (m_keybuf);
>> -  {
>> -    free(m_keybuf);
>> -    m_keybuf = 0;
>> -  }
>> +  free(m_keybuf);
>> +  m_keybuf = 0;
>
> how about changing NULL to 0 in all the other places?
> http://www.stroustrup.com/bs_faq2.html#null
>
> nevertheless, if you like to change it go ahead.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/xbmc/xbmc/pull/4652/files#r12311524